[FLINK-14729] Multi-topics consuming from KafkaTableSource

• Type: New Feature
• Status: Closed
• Priority: Not a Priority
• Resolution: Duplicate
• Component/s: Connectors / Kafka
• Labels: auto-deprioritized-major auto-deprioritized-minor features pull-request-available
• Flags: Important
• Assignee: Unassigned
• Reporter: Leo Zhang
• Created: 13/Nov/19 01:09
• Updated: 16/Oct/23 06:52
• Resolved: 16/Oct/23 06:52

Description

Hi, all. I propose a new functionality of KafkaTableSource which can consume multiple topics at the same time. 
Design plan

• Add a new constructor in KafkaTableSource which accepts topics with List type as one parameter.
• Modify the existed one which only accepts one topic as string type to call the proposed one to finish the instantiation. That is to say, wrap this topic in a list and pass it to the multi-topics-consuming constructor.
• Modify the overridden method createKafkaConsumer in KafkaTableSource to pass topics as List instead of String.
• Replace the field topic with topics as List type in  KafkaTableSourceBase and modify every place using topic with topics. So we just need to modify the constructor KafkaTableSourceBase, method getDataStream, and equals and hashCode.

Test plan

There is less to do as KafkaTableSource is based on FlinkKafkaConsumer which already supports consuming multiple topics and is tested well. Of course, we can easily add further more tests if needed.

 

So what's your opinion?

Issue Links

• is duplicated by: FLINK-18449 Make topic discovery and partition discovery configurable for FlinkKafkaConsumer in Table API
• is superceded by: FLINK-32197 FLIP 246: Dynamic Kafka Source
• links to: GitHub Pull Request #10343
• links to: GitHub Pull Request #10354