ztzg commented on pull request #1644: URL: https://github.com/apache/zookeeper/pull/1644#issuecomment-888947183
@eolivelli wrote: > This is an API change (about the Metrics Provider API) and in my opinion it is better to not pick it on a released branch, in which the API must be stable. > > So I would keep this only on master branch You're right, of course; the change to `MetricsContext` would break third-party metrics providers. @li4wang wrote: > The conflicts have been resolved. Thanks! Looking into it now… -- 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]
