MartijnVisser commented on PR #89:
URL: 
https://github.com/apache/flink-connector-kafka/pull/89#issuecomment-2034536889

   > There are existing tests covering this class and exactly this line of code 
and I also verified it pass. I'm happy to add more tests to ensure the user 
code class loader is being used.
   
   I don't think that's correct, because if there were existing tests I would 
have expected those to fail because of your change. 


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