wuchong commented on a change in pull request #9362: [FLINK-13354] [docs] Add
documentation for how to use blink planner
URL: https://github.com/apache/flink/pull/9362#discussion_r310926794
##########
File path: docs/dev/table/common.md
##########
@@ -27,6 +27,18 @@ The Table API and SQL are integrated in a joint API. The
central concept of this
* This will be replaced by the TOC
{:toc}
+Main Differences Between the Two Planners
+-----------------------------------------
+
+1. The Blink planner does not support `BatchTableSource`, only
`StreamTableSource`. Blink treats batch jobs a special case of streaming. As
such, batch jobs will not be translated into `DateSet` programs but translated
into `DataStream` programs, the same as the streaming jobs.
+2. The Blink planner does not support `ExternalCatalog` which is deprecated.
+3. The implementations of `FilterableTableSource` for the Flink planner and
the Blink planner are incompatible. The Flink planner will push down
`PlannerExpression`s into `FilterableTableSource`, while the Blink planner will
push down `Expression`s.
+4. String based key-value config options (defined in `OptimizerConfigOptions`
and `ExecutionConfigOptions`) are only used for the Blink planner.
Review comment:
We can use a link to the configuration page instead of point out the
classes. The configuration page is located in `docs/dev/table/config.md`
----------------------------------------------------------------
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