boyuanzz commented on pull request #13710:
URL: https://github.com/apache/beam/pull/13710#issuecomment-763088174


   > Don't you think it will make sense to add a IT that test this 
functionality against "real" (see KafkaIOIT) Kafka cluster as well?
   
   I think `ReadFromKafkaDoFnTest` gives a good coverage and it may not be easy 
to add such an IT within current framework(it might be easier with Kafka Admin 
API but I need to have a try on that). I would prefer to have ITs in one 
separated PR later. What do you think?


----------------------------------------------------------------
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:
[email protected]


Reply via email to