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

   [In this PR](https://github.com/apache/kafka/pull/20334), we added some 
validation checks for the connect config, such as ensuring that `plugin.path` 
cannot be empty.
   
    However, currently, Connect first loads the plugin and then creates the 
configuration. Even if `plugin.path` is empty, it still attempts to load the 
plugin first, and then throws an exception when creating the configuration. 
   
   The approach should be to first create a configuration to validate that the 
config meet the requirements, and then load the plugin only if the validation 
passes. This allows for early detection of problems and avoids unnecessary 
plugin loading processes.


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