TheYorouzoya opened a new pull request, #8043: URL: https://github.com/apache/gravitino/pull/8043
<!-- 1. Title: [#8011] Improvement(LockManager): Code in LockManager.java is incorrect --> ### What changes were proposed in this pull request? This PR fixes a bug in the checkDeadLock() function inside core/src/main/java/org/apache/gravitino/lock/LockManager.java. The code is modified to correctly format each thread's timestamp inside the forEach loop. ### Why are the changes needed? Previously, the deadlock logging in LockManager was trying to format the entire map instead of the actual timestamp, which was leading to an IllegalArgumentException when checking for long-held locks. Fix: #8011 ### Does this PR introduce _any_ user-facing change? No. ### How was this patch tested? Locally, with a unit test provided in the original issue. Since the fix basically amounts to fixing a typo rather than changing any core functionality, I've opted to not include the test with the PR. -- 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]
