[FLINK-12734] remove getVolcanoPlanner method from FlinkOptimizeContext and RelNodeBlock does not depend on TableEnvironment

• Type: Improvement
• Status: Closed
• Priority: Major
• Resolution: Implemented
• Fix Version/s: 1.9.0
• Component/s: Table SQL / Planner
• Labels: pull-request-available
• Assignee: godfrey he
• Reporter: godfrey he
• Created: 05/Jun/19 03:26
• Updated: 06/Jun/19 03:27
• Resolved: 06/Jun/19 03:27

Description

there are two improvements:
1. remove getVolcanoPlanner method from FlinkOptimizeContext.
VolcanoPlanner limits that the planer a RelNode tree belongs to and the VolcanoPlanner used to optimize the RelNode tree should be same instance. (see: VolcanoPlanner#registerImpl)
so, we can use planner instance in RelNode's cluster directly instead of planner from getVolcanoPlanner method in FlinkOptimizeContext.

2. RelNodeBlock does not depend on TableEnvironment
In RelNodeBlock, only TableConfig is used.

Issue Links

• is a child of: FLINK-11488 Add a basic Blink planner framework
• links to: GitHub Pull Request #8619