MartijnVisser commented on a change in pull request #18353:
URL: https://github.com/apache/flink/pull/18353#discussion_r796397547
##########
File path: docs/content/docs/dev/table/overview.md
##########
@@ -40,8 +40,8 @@ and later use the DataStream API to build alerting based on
the matched patterns
## Table Program Dependencies
-Depending on the target programming language, you need to add the Java or
Scala API to a project
-in order to use the Table API & SQL for defining pipelines.
+Depending on the target programming language, you need to add the Table API to
a project in order to
+use the Table API & SQL for defining data pipelines.
Review comment:
I would +1 removal of these tabs here. The overview page for Table
should be comparable to the current DataStream Overview, so an overview of the
Table APIs capabilities etc. We've moved the setup to the project configuration
page, which fits better. Python is kind of a weird one with its own
requirements, so for now better to link to the Python.
So instead of having a section Table Program Dependencies, perhaps just a
link to the project configuration page with a text like "Check out the project
configuration for setting up your environment so you can start building a Table
API program"? (But then in nicer phrasing 😅 )
--
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]
