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

Reply via email to