alexr17 opened a new pull request, #13395: URL: https://github.com/apache/hudi/pull/13395
### Change Logs Some code paths seem to directly call close() on the `LockManager` rather than just "unlock". This makes sense, especially if there is a shutdown occurring. However, when we close the `StorageBasedLockProvider` we explicitly remove the shutdown hook. If there is already a shutdown occurring this fails with an `IllegalStateException` and the lock does not get released properly. Therefore we will catch this exception and proceed with normal `close()` logic. ### Impact Fixes a race condition when shutting down the JVM in `StorageBasedLockProvider` ### Risk level (write none, low medium or high below) Low ### Documentation Update N/A ### Contributor's checklist - [ ] Read through [contributor's guide](https://hudi.apache.org/contribute/how-to-contribute) - [ ] Change Logs and Impact were stated clearly - [ ] Adequate tests were added if applicable - [ ] CI passed -- 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]
