infoverload commented on a change in pull request #16871: URL: https://github.com/apache/flink/pull/16871#discussion_r701960875
########## File path: docs/content/docs/dev/table/data_stream_api.md ########## @@ -725,11 +732,17 @@ to an insert-only changelog stream. {{< hint info >}} Since batch processing can be considered as *a special case of stream processing*, we recommend implementing a streaming pipeline first as it is the most general implementation for both bounded and unbounded data. + +In theory, a streaming pipeline can execute all operators. However, in practice, some operations might +not make much sense as they would lead to ever-growing state and are therefore not supported. A global +sort would be an example that is only available in batch mode. Simply put: it should be possible to +run a working streaming pipeline in batch and but not necessarily vice versa. Review comment: ```suggestion run a working streaming pipeline in batch mode but not necessarily vice versa. ``` ########## File path: docs/content/docs/dev/table/data_stream_api.md ########## @@ -2695,7 +2708,7 @@ correctly via the DataStream API's reflective type extraction facilities. If the schema. {{< hint warning >}} -Many times the DataStream API is unable to extract a more specific `TypeInformation` based on reflection. +Many times the DataStream API is unable to extract a more specific `TypeInformation` based on [reflection](https://www.oracle.com/technical-resources/articles/java/javareflection.html). Review comment: ```suggestion The DataStream API is not always able to extract a more specific `TypeInformation` based on [reflection](https://www.oracle.com/technical-resources/articles/java/javareflection.html). ``` -- 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: issues-unsubscr...@flink.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org