wuchong commented on issue #11495: [FLINK-16125][kafka] Make property 
zookeeper.connect required for kafka 0.8 connector and optional for other 
versions
URL: https://github.com/apache/flink/pull/11495#issuecomment-603255833
 
 
   Thanks @PatrickRen for the contribution. Could you add a test in 
`KafkaTableSourceSinkFactoryTestBase` to verity 0.9+ can work wiithout 
`zookeeper.connect`, however 0.8 should fail? 

----------------------------------------------------------------
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.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services

Reply via email to