wenlong88 commented on pull request #17924: URL: https://github.com/apache/flink/pull/17924#issuecomment-986570154
hi, @AHeise thanks for the review. The description would be used in both DataStream and SQL/Table API according to the FLIP discussion, would be used in JSON plan and finally affects content in Web UI, the related change is in [#17938](https://github.com/apache/flink/pull/17938). Improvement on DataStream operators will be in following up issues such as https://issues.apache.org/jira/browse/FLINK-25074. About the naming and whether we need to expose API in DataStream, I think we have reached a consensus in the FLIP discussion, We are trying to introduce descriptions so that we can simplify the names. This is worth in both SQL and DataStream API IMO. Do you have any more suggestion or comments about this? It would be great if there is some better idea. I have updated java docs and addressed other comments, please check again, thanks. -- 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]
