deepdave98 commented on issue #4187:
URL: 
https://github.com/apache/iceberg-python/issues/4187#issuecomment-6749587125

   I have a fix for this ready: `create_branch` and `create_tag` now raise 
`ValueError("Ref x already exists")` when the name is taken, including by an 
earlier operation in the same chain, and `remove_branch` / `remove_tag` raise 
for a missing name, for the wrong kind of ref, and for `main`, with the same 
messages as Java's `UpdateSnapshotReferencesOperation`. Unit tests cover each 
case and the script in this issue now raises on both re-creates. Opening the PR 
now; @tanmayrauth, if you already have a branch for this, say so and I will 
defer to it.
   


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