sjwiesman commented on a change in pull request #15144:
URL: https://github.com/apache/flink/pull/15144#discussion_r592508065
##########
File path: docs/content/docs/dev/table/legacy_planner.md
##########
@@ -35,8 +35,10 @@ The primary reason to continue using the legacy planner is
[DataSet]({{< ref "do
{{< hint warning >}}
If you are not using the Legacy planner for DataSet interop, the community
strongly
-encourages you to consider the modern table planner.
-The legacy planner will be dropped at some point in the future.
+encourages you to consider the modern table planner. Both batch and stream
processing pipelines
+can be expressed in the unified `TableEnvironment`.
+
+The legacy planner is deprecated and will dropped in Flink 1.14.
Review comment:
```suggestion
** The legacy planner is deprecated and will be dropped in Flink 1.14.**
```
##########
File path: docs/content/docs/dev/table/legacy_planner.md
##########
@@ -35,8 +35,10 @@ The primary reason to continue using the legacy planner is
[DataSet]({{< ref "do
{{< hint warning >}}
If you are not using the Legacy planner for DataSet interop, the community
strongly
-encourages you to consider the modern table planner.
-The legacy planner will be dropped at some point in the future.
+encourages you to consider the modern table planner. Both batch and stream
processing pipelines
+can be expressed in the unified `TableEnvironment`.
+
+The legacy planner is deprecated and will dropped in Flink 1.14.
Review comment:
Also please update `docs/content.zh/docs/dev/table/legacy_planner.md`
##########
File path: flink-table/flink-table-planner/pom.xml
##########
@@ -32,7 +32,8 @@ under the License.
<description>
This module bridges Table/SQL API and runtime. It contains
all resources that are required during pre-flight and runtime
- phase.
+ phase. NOTE: This module is deprecated and will be removed in
Flink 1.14.
+ Please upgrade to the new planner instead.
Review comment:
```suggestion
Please upgrade to the new planner.
```
----------------------------------------------------------------
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]