You are viewing a plain text version of this content. The canonical link for it is here.

Posted to issues@flink.apache.org by GitBox <gi...@apache.org> on 2022/01/05 10:19:24 UTC

[GitHub] [flink] fapaul commented on pull request #18145: [FLINK-25368][connectors/kafka] Substitute KafkaConsumer with AdminClient when getting offsets

fapaul commented on pull request #18145:
URL: https://github.com/apache/flink/pull/18145#issuecomment-1005555912


   I agree with @lindong28 if we want to change the `OffsetsInitializer#PartitionOffsetsRetriever` we need a FLIP. In this case, there might be also a different alternative. We could introduce a second internal interface extending the existing one and implement the `list` functionality. AFAICT the list function is only used internally by the enumerator. WDYT?


-- 
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: issues-unsubscribe@flink.apache.org

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