dianfu commented on a change in pull request #11375: [FLINK-16538][python][docs] Restructure Python Table API documentation URL: https://github.com/apache/flink/pull/11375#discussion_r392040809
########## File path: docs/dev/table/python/index.md ########## @@ -0,0 +1,35 @@ +--- +title: "Python Table API" +nav-id: python_tableapi +nav-parent_id: tableapi +nav-pos: 100 Review comment: I'm more tend to the current location for the doc of "Python Table API" for the following reasons: - The documentation about "Table API" is about the operations in the Table class. However, the documentation about "Python Table API" is more general, it includes everything about the Python language support about the table module. So I think there is no direct relationship between "Table API" and "Python Table API". Maybe we should rename it to something else? - The documentation about "Python Table API" is about the Python language support about the table module and each of the items at the beginning is about one specific functionality about table and so personally I think putting "Python Table API" at the end makes more sense. ---------------------------------------------------------------- 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: [email protected] With regards, Apache Git Services
