Add todo on cluster lock (#34133)

apache · Dec 24, 2024 · d187256 · d187256
1 parent 972eabd
commit d187256
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 0 deletions.
diff --git a/...pache/shardingsphere/proxy/backend/handler/distsql/ral/updatable/LockClusterExecutor.java b/...pache/shardingsphere/proxy/backend/handler/distsql/ral/updatable/LockClusterExecutor.java
@@ -44,6 +44,7 @@ public void executeUpdate(final LockClusterStatement sqlStatement, final Context
         checkAlgorithm(sqlStatement);
         LockContext lockContext = contextManager.getComputeNodeInstanceContext().getLockContext();
         GlobalLockDefinition lockDefinition = new GlobalLockDefinition(new ClusterLock());
+        // TODO should configured in SQL Statement
         if (lockContext.tryLock(lockDefinition, 3000L)) {
             try {
                 checkState(contextManager);

diff --git a/...che/shardingsphere/proxy/backend/handler/distsql/ral/updatable/UnlockClusterExecutor.java b/...che/shardingsphere/proxy/backend/handler/distsql/ral/updatable/UnlockClusterExecutor.java
@@ -39,6 +39,7 @@ public void executeUpdate(final UnlockClusterStatement sqlStatement, final Conte
         checkState(contextManager);
         LockContext lockContext = contextManager.getComputeNodeInstanceContext().getLockContext();
         GlobalLockDefinition lockDefinition = new GlobalLockDefinition(new ClusterLock());
+        // TODO should configured in SQL Statement
         if (lockContext.tryLock(lockDefinition, 3000L)) {
             try {
                 checkState(contextManager);