kaisun2000 commented on pull request #1119:
URL: https://github.com/apache/helix/pull/1119#issuecomment-650387322


   Sync() failed, the rc code would tell you the reason. Similar like 
KeeperException. 
   For this case, I think we can do it,
   
   Ok: proceed
   retryable: connectionloss, we retry
   
   Others, including session expiration, just eat it, log it and stop here. 
   
   


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

For queries about this service, please contact Infrastructure at:
[email protected]



---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to