[GitHub] [curator] tisonkun commented on pull request #478: CURATOR-688. SharedCount will be never updated successful when version of ZNode is overflow.

["tisonkun (via GitHub)" <gi...@apache.org>]

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

   > TBH, I am not familiar with implement about Curator, so would you mind to list some demo code snippet about configuration if this solution could be approved.
   
   Somehow you can copy the shared count implementation..It's not quite a lot of code.
   
   Since this requirement is quite customized, I'm afraid that it's not suitable to hack into Curator. The root cause is integer overflow that is backed by ZK. 
   
   Also, I don't understand actually how this patch "fixes" the issue. You add more strict condition to exit but the original issue is hang?


-- 
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: commits-unsubscribe@curator.apache.org

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