jiajunwang commented on PR #2293: URL: https://github.com/apache/helix/pull/2293#issuecomment-1332558081
I don't have any concern regarding the change. Just to provide some more context: One question we discussed before is what to do if the operation is timed out? If users are going to manually retry anyway, then just keep retrying seems to be a reasonable choice. Actually inside Helix control plane, retrying is the only option, IMHO. In other systems, the timeout error state does not necessarily stop the retry process. But it seems to be too complicated for Helix. -- 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] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
