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]


Reply via email to