FlyingZC opened a new issue, #35041:
URL: https://github.com/apache/shardingsphere/issues/35041

   ## Feature Request
   
   Refactor the lock implementation in GlobalClockTransactionHook.
   
   1.  Avoid using registry-based locks (e.g., ZooKeeper) because acquiring a 
lock might timeout in beforeCommit, but the afterCommit could still attempt to 
release it. Eliminate unnecessary unlock operations. 
   
   Corresponding FIXME:
   ```
   FIXME: If timeout occurs when a lock is required but TSO is not assigned, 
the commit will still proceed. The solution is to replace the registry center's 
lock with a Redis-based lock implementation.
   ```
   
   2. Move the locking logic to the TSO component (e.g., implement 
locking/unlocking via Redis's native lock in RedisTSOProvider). Corresponding 
TODO:
   
   ```
   // TODO: Use Redis lock instead of registry center's lock.
   ```
   
   


-- 
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: 
notifications-unsubscr...@shardingsphere.apache.org.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Reply via email to