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

Reply via email to