chia7712 commented on pull request #9284:
URL: https://github.com/apache/kafka/pull/9284#issuecomment-696082528


   ```
   Build / JDK 11 / 
kafka.api.ConsumerBounceTest.testRollingBrokerRestartsWithSmallerMaxGroupSizeConfigDisruptsBigGroup
   ```
   
   only jdk 11 get failed test. It is unrelated to this patch.
   
   @hachikuji It should be ready to be merged. BTW, this patch does break the 
behavior of altering non-reconfigurable configs of existent listeners as we 
didn't throw exception before. Not sure whether we should do more for this 
change. Maybe It would be better to add extra comment to git comment when 
merging this patch. For example:
   
   `
   this patch introduces an incompatible behavior change. Altering 
non-reconfigurable configs of existent listeners causes exception instead of 
being a NOP 
   `


----------------------------------------------------------------
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.

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


Reply via email to