rambleraptor commented on code in PR #18264:
URL: https://github.com/apache/iceberg/pull/18264#discussion_r4211689608
##########
core/src/main/java/org/apache/iceberg/TableMetadata.java:
##########
@@ -63,6 +63,8 @@ public class TableMetadata implements Serializable {
static final int INITIAL_SORT_ORDER_ID = 1;
static final int INITIAL_SCHEMA_ID = 0;
static final int INITIAL_ROW_ID = 0;
+ // field IDs above this are reserved by the spec for metadata columns
+ static final int MAX_FIELD_ID = 2147483447;
Review Comment:
I can do that. I was originally doing that on purpose (Integer.MAX_VALUE may
be a different number on other implementations), but I'm very unopinionated.
--
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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
