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

Reply via email to