rdsarvar commented on code in PR #11368:
URL: https://github.com/apache/iceberg/pull/11368#discussion_r1808958390
##########
api/src/main/java/org/apache/iceberg/UpdatePartitionSpec.java:
##########
@@ -133,4 +133,16 @@ default UpdatePartitionSpec addNonDefaultSpec() {
throw new UnsupportedOperationException(
this.getClass().getName() + " doesn't implement addNonDefaultSpec()");
}
+
+ /**
+ * Explicitly providing the partition spec that we would like to use. When a
spec has been
+ * provided then modifications should not be done afterwards through this
class.
+ *
+ * @param newSpec partition spec to override the builder use during commit
+ * @return this for method chaining names.
+ */
+ default UpdatePartitionSpec useSpec(PartitionSpec newSpec) {
Review Comment:
This feels... hacky. Question for folks reading this would be if it's
overkill to support a `AddPartitionSpec` operation instead of relying on the
`UpdatePartitionSpec`
--
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
