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:
us...@infra.apache.org


With regards,
Apache Git Services

Reply via email to