RussellSpitzer commented on code in PR #12593: URL: https://github.com/apache/iceberg/pull/12593#discussion_r2011060176
########## core/src/main/java/org/apache/iceberg/TableMetadata.java: ########## @@ -57,8 +57,6 @@ 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; - static final boolean DEFAULT_ROW_LINEAGE = false; - static final int MIN_FORMAT_VERSION_ROW_LINEAGE = 3; Review Comment: I still kind of like having this as a CONSTANT at the top of the file so we can be clear when we are gating something it's because of row_lineage and not generic v3 things. But either way is fine with me -- 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