[
https://issues.apache.org/jira/browse/FLINK-12734?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
godfrey he updated FLINK-12734:
-------------------------------
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.
was:
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
{{getVolcanoPlanner}} from {{FlinkOptimizeContext}}.
2. {{RelNodeBlock}} does not depend on {{TableEnvironment}}
In {{RelNodeBlock}}, only {{TableConfig}} is used.
> remove getVolcanoPlanner method from FlinkOptimizeContext and RelNodeBlock
> does not depend on TableEnvironment
> --------------------------------------------------------------------------------------------------------------
>
> Key: FLINK-12734
> URL: https://issues.apache.org/jira/browse/FLINK-12734
> Project: Flink
> Issue Type: Improvement
> Components: Table SQL / Planner
> Reporter: godfrey he
> Assignee: godfrey he
> Priority: Major
> Labels: pull-request-available
> Time Spent: 10m
> Remaining Estimate: 0h
>
> 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.
--
This message was sent by Atlassian JIRA
(v7.6.3#76005)
