AHeise commented on PR #115:
URL: 
https://github.com/apache/flink-connector-kafka/pull/115#issuecomment-2320739887

   > LGTM although I am slightly confused if users could have used the classes 
removed by this PR
   
   It was originally used by the universal producer (FlinkKafkaProducer) and 
the FlinkKafkaProducer01(1). It was already refactored some years ago and only 
left in to be able to read the serialized state of that old producer.
   
   The base class is annotated as @Internal. The `FlinkKafkaProducer011` is not 
marked at all but the java doc clearly states that this is just for migration.
   
   So I have a hard time to imagine anyone using this class and being surprised 
that it doesn't work anymore.


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