dimas-b commented on code in PR #5010:
URL: https://github.com/apache/iceberg/pull/5010#discussion_r918330963


##########
core/src/main/java/org/apache/iceberg/SnapshotProducer.java:
##########
@@ -116,6 +117,38 @@ public ThisT scanManifestsWith(ExecutorService 
executorService) {
     return self();
   }
 
+  @Override
+  public ThisT toBranch(String branch) {
+    throw new UnsupportedOperationException("Performing operations on a branch 
is currently not supported");

Review Comment:
   nit: the error wording looks a bit confusing to me. Are we planning to 
support the `toBranch` method in this class later or is it not envisioned to be 
supported on this particular class at all? thx!



-- 
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