[CALCITE-189] Add a way to specify list of TraitDefs that will be used by planner when...

• Type: Bug
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Labels: github-import
• Assignee: Unassigned
• Reporter: GitHub Import
• Created: 18/Jun/14 08:30
• Updated: 18/Jun/14 08:30
• Resolved: 18/Jun/14 08:30

Description

Hi Julian,

This is the code change to support user to explicitly specify the list of RelTraitDef to register with VolcalnoPlanner, when user call Frameworks.getPlanner(). If user provides such list, then, the planner will de-register any existing RelTraitDef, before register the RelTraitDef in the provided list.

This code change has two purposes. 1) User could specify the list of RelTraitDef, 2) User could specify the order of importance of each RelTraitDef. (The order of importance will determine the sequence call of RelTraitDef.convert() in VolcalnoPlanner.changeTraitsUsingConvert() method. )

I also added a unit test case in PlannerTest:testPlanWithExplicitTraitDefs(). This test is almost same as PlannerTest:testPlan(), except the former uses an explicitly specified list of RelTraitDef, when creates the planner.

---------------- Imported from GitHub ----------------
Url: https://github.com/julianhyde/optiq/pull/189
Created by: jinfengni
Labels:
Created at: Mon Mar 17 22:48:05 CET 2014
State: closed