thswlsqls opened a new pull request, #17084:
URL: https://github.com/apache/iceberg/pull/17084

   
   Closes #17081
   
   ## Summary
   
   - The "Replacing and fast forwarding" example called `replaceBranch(tag, 4)` 
on `tag = "audit-tag"`, a tag, which throws `IllegalArgumentException` at 
runtime (`UpdateSnapshotReferencesOperation.replaceBranch` requires the target 
ref to be a branch).
   - Switched to `replaceTag(tag, 4)`, the API `ManageSnapshots` provides for 
replacing tags.
   - Matches the same file's usage of `"audit-tag"` as a tag two examples 
earlier (`useRef("audit-tag")`).
   
   ## Testing done
   
   - Docs-only change, no behavior change — no test added. Verified against 
`core/src/main/java/org/apache/iceberg/UpdateSnapshotReferencesOperation.java` 
(`replaceBranch` guard) and 
`api/src/main/java/org/apache/iceberg/ManageSnapshots.java` (`replaceTag` 
signature).
   
   
   


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