danielcweeks commented on code in PR #13123:
URL: https://github.com/apache/iceberg/pull/13123#discussion_r2124402347


##########
api/src/test/java/org/apache/iceberg/TestHelpers.java:
##########
@@ -54,7 +54,7 @@ public class TestHelpers {
 
   private TestHelpers() {}
 
-  public static final int MAX_FORMAT_VERSION = 3;
+  public static final int MAX_FORMAT_VERSION = 4;

Review Comment:
   The main thing is that this is necessary for anyone looking to do 
exploratory work on the next version of the spec.  v3 is adopted and future 
work (anything that's backwards in compatible) needs to be built against v4.  
Since there are a number of metadata related proposals, we need a way to build 
PRs that don't involve everyone rebuilding this same functionality as an 
initial step.



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscr...@iceberg.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscr...@iceberg.apache.org
For additional commands, e-mail: issues-h...@iceberg.apache.org

Reply via email to