Hexiaoqiao commented on PR #478:
URL: https://github.com/apache/curator/pull/478#issuecomment-1722674942

   @tisonkun Glad to see you here.
   
   > While it can be better for ZK to jump from version -2 to 0 so that in an 
overflow loop we don't handle -1 hole, the restart and get versioned value = -1 
remains and set data concurrently doesn't hurt as long as we read the updated 
data finally.
   
   It will be more smooth while ZK to skip version -1, Strong +1. From my first 
thought, it would be the next step after we fix at Curator side. Now it is 
great to see that other guys already try to push this forward. 
   For ZOOKEEPER-4743, I just suggest that we should consider to solve 
dataVersion/aclVersion/cversion 32-integer overflow at once.
   Back to this PR, what should we do before ZOOKEEPRER-4743 check in, just 
wait ZOOKEEPER-4743 to be ready or other ways? Thanks.


-- 
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]

Reply via email to