Re: [PR] [common] Fix snapshot and tag reads failing when the file is deleted concurrently [paimon]

["sundapeng (via GitHub)" <gi...@apache.org>]

sundapeng commented on PR #9851:
URL: https://github.com/apache/paimon/pull/9851#issuecomment-5683623986

   Thanks, that makes sense. `Tag`, `Changelog` and `SchemaManager` have the same gap, and all of them read through `FileIO#readFileUtf8`, so I moved the check there: when a read fails with an `IOException` other than `FileNotFoundException` and the file no longer exists, it throws `FileNotFoundException`. `readOverwrittenFileUtf8` already does a similar `exists()` check for hint files.
   
   I kept it in the default method instead of the OSS and Jindo FileIOs, because those can't tell a missing file from other failures either: hadoop-aliyun drops the OSS error code and fails with `Null IO stream`.
   


-- 
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: issues-unsubscribe@paimon.apache.org

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