gharris1727 commented on pull request #8259:
URL: https://github.com/apache/kafka/pull/8259#issuecomment-788250316


   Now that the bug is fixed, some comments are explicitly incorrect. For 
example:
   ```
           // 1. Lock the PluginClassLoader (via PluginClassLoader::loadClass)
           // This behavior is specific to the JVM, not the classloader 
implementation
   ```
   And a lot have the wrong intent, or are misleading. Do you think we should 
fix some or all of these, or leave them as a detailed description of how the 
bug _did_ happen when it existed?


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