lh0156 opened a new pull request, #23030:
URL: https://github.com/apache/kafka/pull/23030

   ## Summary
   
   Connector validation can generate a ConfigInfo entry without a 
ConfigValueInfo when a connector defines a key that the plugin does not return. 
AbstractHerder.maybeAddConfigErrors previously dereferenced that null value and 
replaced the original validation response with an NPE.
   
   This change skips entries without a ConfigValueInfo while retaining every 
error from populated values. Focused tests cover errors before and after the 
null entry, callback completion, and the no-error path.
   
   ## Testing
   
   - TDD RED: the focused regression class failed 3 of 6 tests with the 
baseline NPE.
   - TDD GREEN: `./gradlew :connect:runtime:test --tests 
org.apache.kafka.connect.runtime.AbstractHerderConfigErrorTest --no-build-cache 
--console=plain` (6 tests passed)
   - A full `:connect:runtime:test` run reached two unrelated timing-sensitive 
`SynchronizationTest` timeouts; both tests passed when rerun in isolation.
   - The affected checkstyle and SpotBugs tasks passed.
   - `git diff --check`
   
   Jira: https://issues.apache.org/jira/browse/KAFKA-20693


-- 
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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to