amogh-jahagirdar commented on code in PR #10736:
URL: https://github.com/apache/iceberg/pull/10736#discussion_r1690525854
##########
api/src/main/java/org/apache/iceberg/UpdatePartitionSpec.java:
##########
@@ -122,4 +122,13 @@ public interface UpdatePartitionSpec extends
PendingUpdate<PartitionSpec> {
* change conflicts with other additions, removals, or renames.
*/
UpdatePartitionSpec renameField(String name, String newName);
+
+ /**
+ * Sets whether the new partition spec will be set as the default partition
spec for the table,
+ * the default behavior is to do so.
+ *
+ * @param value whether to set the new partition spec as default
+ * @return this for method chaining
+ */
+ UpdatePartitionSpec setAsDefault(Boolean value);
Review Comment:
Yeah couldn’t think of a good name for a no arg method so I discarded it
initially but another one came to mind: addWithoutSettingCurrent (maybe a bit
verbose)
--
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
