alpinegizmo commented on pull request #18055:
URL: https://github.com/apache/flink/pull/18055#issuecomment-1026696762


   Given that we now have a working file source for both streaming and batch in 
flink master, I'm in favor of reworking this tutorial to use that connector -- 
thus dropping flink-faker. A relatively small file should be enough -- and it 
could be bootstrapped from flink-faker.
   
   I think it would also be quite useful (from a pedagogical perspective) to 
extend the top-n section so that it does the same query in both batch and 
streaming modes. 


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

For queries about this service, please contact Infrastructure at:
[email protected]


Reply via email to