[GitHub] [flink] knaufk commented on issue #8903: [FLINK-12747][docs] Getting Started - Table API Example Walkthrough

[GitBox <gi...@apache.org>]

knaufk commented on issue #8903: [FLINK-12747][docs] Getting Started - Table API Example Walkthrough
URL: https://github.com/apache/flink/pull/8903#issuecomment-506153339
 
 
   Thanks a lot @sjwiesman  for the contribution! A few general thoughts: 
   
   * I really like that it's not the typical generic "wordcount"/"windowed wordcount"/etc.
   
   * I would be in favor of simply adding this as an example to `flink-examples-table` without a maven archtype. Firstly, we already have quickstart modules to create program skeletons. Adding an additional option for the walkthroughs adds extra burden in terms of ongoing maintenance and might confuse users. Secondly, in my opinion a "code walkthrough" should focus on explainoing the code of an example so that I can easily follow it. I don't necessarily expect the utilities to bootstrap a new program here. (We should have a separate dedicated subsection in the "Getting Started" section for the quickstarts/project setup.)
   
   * Personally I think, the example for a code walkthrough should be no-magic and I should be able to understand as much as possible as a new user. Here this means to go without the custom `TableSource`, which is not easy to understand and it is not clear why this can be a streaming and batch source at the same time etc.. Alternatively, one could simply read from a file of transactions (batch case) or read from a message queue (streaming case; maybe provide a minimal data generator). This would also go well with the future vision of "simply replace the source and your job turns into a streaming program", although we are not there yet.
   
   * Do we also want to add SQL to this example (probably as a section in the end implementing the same business logic)? Currently, we don't plan to have dedicated "SQL Walkthrough". Do we need an SQL walkthrough at all?
   
   What do you think? What do others 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:
users@infra.apache.org


With regards,
Apache Git Services