dhruv-pratap commented on code in PR #13903: URL: https://github.com/apache/iceberg/pull/13903#discussion_r2303968811
########## core/src/main/java/org/apache/iceberg/TableProperties.java: ########## @@ -244,6 +244,12 @@ private TableProperties() {} public static final String DELETE_PLANNING_MODE = "read.delete-planning-mode"; public static final String PLANNING_MODE_DEFAULT = PlanningMode.AUTO.modeName(); + public static final String METADATA_PLANNING_MODE = "read.metadata-planning-mode"; Review Comment: I think this ties into @RussellSpitzer 's point he's made above. Let me amend the PR to remove the "Planning mode" complexity and simply swap the implementation. -- 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