[GitHub] [helix] xyuanlu commented on pull request #2385: Add retry policy class and config for operation retry

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

xyuanlu commented on PR #2385:
URL: https://github.com/apache/helix/pull/2385#issuecomment-1453925687

   > Overall LGTM! One general comment, Do we want to name all class as "_reconnect_policy" or "_retry_policy"? From your description and functionality it looks like later one to me.
   
   TFTR. 
   Retry connection policy - how we reconnection in a separate thread when connection lost. Different policies can be defined for this.
   Retry - how we define behavior for retry CRUD operation when connection is lost. We only define timeout for this. 


-- 
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: reviews-unsubscribe@helix.apache.org

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


---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@helix.apache.org
For additional commands, e-mail: reviews-help@helix.apache.org