bbeaudreault opened a new pull request #4285:
URL: https://github.com/apache/hbase/pull/4285
While I was here, I also changed the default scope. Just my opinion, but I
feel like the toString representation is not a good default value because it is
relatively meaningless. The one good thing it does is ensure uniqueness of the
scopes. I changed the default value to be
`${clusterId}-${connection.hashCode}`. This should provide the same uniqueness,
but using clusterId will help users identify metrics in processes that connect
to more than one hbase cluster.
I'm not sure if we'd consider that change to be a compatibility issue. It
only affects the `scope=` part of the JMX MBean objectName. If this is a
concern, I can revert that part.
--
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: issues-unsubscr...@hbase.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
