ascherbakoff commented on code in PR #2720:
URL: https://github.com/apache/ignite-3/pull/2720#discussion_r1410832696
##########
modules/transactions/src/main/java/org/apache/ignite/internal/tx/LockManager.java:
##########
@@ -76,13 +86,21 @@ public interface LockManager {
* @return The waiter.
*/
@TestOnly
- public Waiter waiter(LockKey key, UUID txId);
+ Waiter waiter(LockKey key, UUID txId);
/**
* Returns {@code true} if no locks have been held.
*
* @return {@code true} if no locks have been held.
*/
@TestOnly
- public boolean isEmpty();
+ boolean isEmpty();
+
+ /**
+ * Get parent lock manager.
+ *
+ * @return Parent lock manager in lock hierarchy.
+ */
+ @Nullable
+ LockManager parentLockManager();
Review Comment:
I've removed this method at the cost of additional check during lock
acquiring. For coarse-grained lock (depending on LockKey) it goes to
parentLockManager
--
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]
