[GitHub] [iceberg] HeartSaVioR commented on pull request #1559: Core: Update version-hint.txt atomically

[GitBox <gi...@apache.org>]

HeartSaVioR commented on pull request #1559:
URL: https://github.com/apache/iceberg/pull/1559#issuecomment-704792040


   If I understand correctly, these operations should be applied regardless of existing version file - consider the case when two concurrent writers both see there's no existing version file and try to write to the path directly.
   
   That said, delete -> rename can be (should be?) replaced with rename with overwrite = true. If I'm not missing anything, this ensures last one wins and partial file is not exposed.


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

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@iceberg.apache.org
For additional commands, e-mail: issues-help@iceberg.apache.org