[GitHub] [kafka] xvrl commented on pull request #11201: MINOR: fix mbean tag name ordering in JMX reporter

[GitBox <gi...@apache.org>]

xvrl commented on pull request #11201:
URL: https://github.com/apache/kafka/pull/11201#issuecomment-898649031


   @dajac agree there might be some surprises for users that relied on ordering that might appear constant, but never guaranteed that. Unfortunately, that order would be dependent on a particular JVM's hashmap implementation, which in practice might not vary often, but has changed in the past and could vary by vendor (e.g. Java 8 even called out some changes to the ordering https://docs.oracle.com/javase/8/docs/technotes/guides/collections/changes8.html)
   
   I'll file a ticket so that we can at least include it in the release notes and call it out as one of the things to be aware of when upgrading. Since we never guaranteed tag ordering though, I don't think we need a KIP for this.
   


-- 
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: jira-unsubscribe@kafka.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org