RussellSpitzer commented on code in PR #15656:
URL: https://github.com/apache/iceberg/pull/15656#discussion_r2943202077


##########
core/src/main/java/org/apache/iceberg/SnapshotChanges.java:
##########
@@ -72,11 +73,16 @@ private SnapshotChanges(
    * @return a new Builder
    */
   public static Builder builderFor(Table table) {
-    return new Builder(table.currentSnapshot(), table.io(), table.specs());
+    Builder builder = new Builder(table.io(), table.specs());
+    if (table.currentSnapshot() != null) {

Review Comment:
   You can potentially call this on a branch when no current snapshot exists 
yet 



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