[GitHub] [flink] StephanEwen commented on issue #11092: [FLINK-15999] Extract “Concepts” material from API/Library sections and start proper concepts section

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

StephanEwen commented on issue #11092: [FLINK-15999] Extract “Concepts” material from API/Library sections and start proper concepts section
URL: https://github.com/apache/flink/pull/11092#issuecomment-586403438
 
 
   Thanks, this is a great first step!
   
   Is the plan to merge this with the TODO comments? (I am personally fine with that)
   Or is that a base PR on which you want to base the next ones?
   
   A few minor suggestions:
     - I would remove the "Programming Model (outdated)" section.
     - Instead, having a staring page (maybe for concepts, maybe for the docs as a whole) that show the stack (SQL/Table API / DataStream API / StateFun, those on top of the common runtime)
   
     - I would drop operator state from the introduction. Instead I would mention it in the `DataStream API` as something that is mainly targeted towards connectors.
   
     - What do you think about removing ingestion time from the "timely" docs? We could change it to just a short mention under "event time", or mention it not at all.
   
     - I would also remove the "Asynchronous State Snapshots" section and simply have a comment somewhere that all persistence operations are asynchronous.

----------------------------------------------------------------
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