kbendick commented on code in PR #4546:
URL: https://github.com/apache/iceberg/pull/4546#discussion_r850050979


##########
core/src/main/java/org/apache/iceberg/TableProperties.java:
##########
@@ -75,6 +75,20 @@ private TableProperties() {
    */
   public static final String CURRENT_SNAPSHOT_TIMESTAMP = 
"current-snapshot-timestamp-ms";
 
+  /**
+   * Reserved table property for default partition spec.
+   * <p>
+   * This reserved property is used to store the default partition spec.
+   */
+  public static final String DEFAULT_PARTITION_SPEC = "default-partition-spec";

Review Comment:
   Maybe something like “JSON representation of the table’s current configured 
partition spec, which will be used if not overridden for individual writes”. 
Kind of wordy but something along those lines would be helpful for me if 
quickly looking through the JavaDocs etc. Will leave that decision to you 
though.



-- 
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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to