diqiu50 opened a new pull request, #10742:
URL: https://github.com/apache/gravitino/pull/10742

   ### What changes were proposed in this pull request?
   
   Fix the `file:` scheme handler in `FetchFileUtils` to synchronize symlink 
creation and avoid concurrent races. Replace the non-atomic `exists() + 
delete()` in `KerberosClient` with `Files.deleteIfExists()`. Remove the unused 
duplicate `FetchFileUtils` in `catalog-hive` and add comprehensive tests in 
`hive-metastore-common`.
   
   ### Why are the changes needed?
   
   `Files.createSymbolicLink()` throws `FileAlreadyExistsException` when 
multiple threads concurrently fetch the same keytab path, such as when the 
search event listener triggers cascade sync after catalog creation.
   
   Fix: #10741
   
   ### Does this PR introduce _any_ user-facing change?
   
   No.
   
   ### How was this patch tested?
   
   - added unit coverage in `TestFetchFileUtils` including concurrent, 
idempotent, and target-replacement scenarios


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

Reply via email to