GrantPSpencer commented on PR #2759: URL: https://github.com/apache/helix/pull/2759#issuecomment-1947481569
> Any particular reason for adding this API ? @desaikomal For upcoming functionality where multiple actors can put cluster into maintenance mode for different reasons. Example, if cluster is maintenance mode for reason_A, reason_B, and reason_C, the cluster will not fully exit and delete the node until all reasons have been removed. ZKHelixAdmin will have API to update the cluster maintenance signal and it will also delete the znode if there are no more reasons left in it. However, there is a possible race condition where a reason was added after the client retrieves the znode information but before delete is called. This can be avoided by requiring the znode versions to match when we delete. -- 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]
