bob-barrett opened a new pull request, #14721:
URL: https://github.com/apache/kafka/pull/14721

   There are some places in the code where we create `ConfigException` object 
by passing in a message String and an Exception. This is a standard pattern for 
exceptions, but it is incorrect for `ConfigException`, which interprets these 
arguments as a config key and config value. When we do this, the original 
exception is lost, since it is not interpreted as a `Throwable`, which can make 
debugging configuration errors difficult. This PR replaces such uses of 
`ConfigException` with `InvalidConfigurationException`.
   
   ### Committer Checklist (excluded from commit message)
   - [ ] Verify design and implementation 
   - [ ] Verify test coverage and CI build status
   - [ ] Verify documentation (including upgrade notes)
   


-- 
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-unsubscr...@kafka.apache.org

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

Reply via email to