LuciferYang opened a new pull request, #13995: URL: https://github.com/apache/gravitino/pull/13995
### What changes were proposed in this pull request? `HDFSFileSystemProxy` keeps the `KerberosClient` it logs in with and closes it in the `close()` hook that runs after `FileSystem.close()`, which stops the ticket refresh thread. It also closes the client when creating the file system fails after login. ### Why are the changes needed? The client was dropped after login and the hook was empty, so every closed Kerberos HDFS file system left a refresh thread running for the life of the JVM. Fix: #13991 ### Does this PR introduce _any_ user-facing change? No. ### How was this patch tested? `TestHDFSFileSystemProxy.testClosingFileSystemStopsKerberosTicketRefresh` creates a Kerberos proxy with a mocked login, closes the proxied file system twice and checks the client is closed once. -- 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]
