[GitHub] [kafka] yashmayya commented on pull request #14361: KAFKA-15387: Deprecate Connect's redundant task configurations endpoint

["yashmayya (via GitHub)" <gi...@apache.org>]

yashmayya commented on PR #14361:
URL: https://github.com/apache/kafka/pull/14361#issuecomment-1713326240

   Thanks Sagar!
   
   > a test which would have invoked the older end point and validates that the warning log line gets printed
   
   Feels like a bit of overkill, but I guess it doesn't really hurt to add such a test so I've added one. There's currently no integration test class for Connect's REST API - most of the functionality is tested indirectly through other ITs and some of the more specialized ones have their own test classes (`ConnectorRestartApiIntegrationTest`, `ConnectorTopicsIntegrationTest`, `OffsetsApiIntegrationTest` etc.). It didn't seem worth it to introduce a new integration test class just for this small test so I've added it to `ConnectWorkerIntegrationTest` which is the most generic one.


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

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