Uploaded image for project: 'Calcite'
  1. Calcite
  2. CALCITE-189

Add a way to specify list of TraitDefs that will be used by planner when...

    XMLWordPrintableJSON

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: None
    • Component/s: None
    • Labels:

      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

        Attachments

          Activity

            People

            • Assignee:
              Unassigned
              Reporter:
              github-import GitHub Import
            • Votes:
              0 Vote for this issue
              Watchers:
              0 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: