[FLINK-16125] Make zookeeper.connect optional for Kafka connectors

• Type: Improvement
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 1.10.0
• Fix Version/s: 1.10.1, 1.11.0
• Component/s: Connectors / Kafka
• Labels: pull-request-available
• Assignee: Qingsheng Ren
• Reporter: Jiangjie Qin
• Created: 18/Feb/20 02:44
• Updated: 03/May/20 11:10
• Resolved: 03/May/20 11:10

Description

FLINK-14649 accidentally changed the connector option zookeeper.connect from optional to required for all the Kafka connector versions, while it is only required for 0.8. 

The fix would be make it optional again. This does mean that people who are using Kafka 0.8 might miss this option and get an error from Kafka code instead of Flink code, but given that Kafka 0.8 probably has a small user base now and users will still get an error. I think it is fine.

Issue Links

• duplicates: FLINK-17129 【Flink Kafka Connector】The DDL config(connector.properties.zookeeper.connect) could be optioanal
• links to: GitHub Pull Request #11425
• links to: GitHub Pull Request #11495