ivandika3 commented on PR #10228: URL: https://github.com/apache/ozone/pull/10228#issuecomment-4657777034
@ptlrs Thanks for the patch. I just skimmed it quickly. The regex will work, but once it's deployed we need to stick to it. From HDDS-14510, it is better to use the OM ID (instead of OM node ID) to identify a unique OM node (since the client-side and server-side OM node ID configuration can be different). So I think it's better for HDDS-14510 to be addressed first and then we use OM ID for the suggested leader. Please let me know what you think. You can pick up HDDS-14510 if you like. -- 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]
