adoroszlai commented on PR #3820: URL: https://github.com/apache/ozone/pull/3820#issuecomment-1308277548
@captainzmc, there is no need for further review of this PR, it is good for `master`, can be merged. Backporting it to 1.3 is a different question. I don't see dev communication that this is blocking the release. I see a [request to include it in the release if possible](https://lists.apache.org/thread/3701bl0rmc13hckokt38kbvcyw1h7jvz). I'm not in favor of making such a change so late in the 1.3 release cycle. Enabling OM gRPC is a matter of simple config change, anyone can do it manually. I think it makes sense to first ship it disabled by default, let users enable on an as-needed basis and provide feedback, then enable in the next release based on that. -- 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]
