You are viewing a plain text version of this content. The canonical link for it is here.

Posted to notifications@zookeeper.apache.org by "kezhuw (via GitHub)" <gi...@apache.org> on 2023/09/23 13:26:44 UTC

[GitHub] [zookeeper] kezhuw commented on pull request #2059: ZOOKEEPER-4706: Support 64-bit sequential nodes using zxid

kezhuw commented on PR #2059:
URL: https://github.com/apache/zookeeper/pull/2059#issuecomment-1732318519

   > IIRC zxid only use the least significant 32 bit as sequential id, so it's still overflow with Integer.MAX_VALUE bound.
   
   No, overflow of the lower 32-bits means re-election.
   https://github.com/apache/zookeeper/blob/b31f776471fef79ab161f416d58367bdffaf37a9/zookeeper-server/src/main/java/org/apache/zookeeper/server/quorum/Leader.java#L1247-L1255


-- 
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-unsubscribe@zookeeper.apache.org

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