rdblue commented on code in PR #12596: URL: https://github.com/apache/iceberg/pull/12596#discussion_r2008116032
########## core/src/main/java/org/apache/iceberg/MetadataColumns.java: ########## @@ -93,21 +93,34 @@ private MetadataColumns() {} "_commit_snapshot_id", Types.LongType.get(), "Commit snapshot ID"); + public static final NestedField ROW_ID = + NestedField.optional( + Integer.MAX_VALUE - 107, "_row_id", Types.LongType.get(), "Row ID for row lineage"); + public static final NestedField LAST_UPDATED_SEQUENCE_NUMBER = + NestedField.optional( + Integer.MAX_VALUE - 108, + "_last_updated_sequence_number", + Types.LongType.get(), + "Last updated sequence number for row lineage"); Review Comment: "Sequence number when the row was last updated"? I don't know that we want to call out "row lineage" because people that see this may not know what that is. -- 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