kbuci opened a new pull request, #10523: URL: https://github.com/apache/hudi/pull/10523
### Change Logs - Add a new flag to `org.apache.hudi.client.transaction.lock.LockManager` , that is set once the HoodieLockMetrics lock duration timer had started, and is unset after said timer has been ended. This flag is used to prevent closing the lock duration timer if it has not already been started - This is needed to avoid the scenario in https://issues.apache.org/jira/browse/HUDI-7308 , where if an exception is raised while LockManager tries to acquire lock of LockProvider, and the caller then directly calls LockManager::close (in order to ensure lock is released for other users) , another exception ``HoodieException("Timer was not started")` will be thrown, which may "suppress" the original exception encountered when LockManager tried to acquire lock. ### Impact - This should allow users to easily identify/debug issues when an implementation of LockProvider throws an exception during `java.util.concurrent.locks.Lock#tryLock(long, java.util.concurrent.TimeUnit)` ### 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]
