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

In Volcano, populate RelOptRuleCall.nodeInputs for operands of type "any"

    Details

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

      Description

      If you have an "any" operand, it is useful to know all of the siblings of the RelNode that matched the operand. Currently VolcanoRuleCall does not populate RelOptRuleCall.nodeInputs.

        Issue Links

          Activity

          Hide
          julianhyde Julian Hyde added a comment -

          A fix is available in https://github.com/julianhyde/calcite/tree/990-any-siblings. Milinda Lakmal Pathirage, You can incorporate the fix in your branch to fix CALCITE-968, if you wish. Let me know whether it works for you.

          Show
          julianhyde Julian Hyde added a comment - A fix is available in https://github.com/julianhyde/calcite/tree/990-any-siblings . Milinda Lakmal Pathirage , You can incorporate the fix in your branch to fix CALCITE-968 , if you wish. Let me know whether it works for you.
          Hide
          milinda Milinda Lakmal Pathirage added a comment -

          Julian Hyde, fix works.

          Show
          milinda Milinda Lakmal Pathirage added a comment - Julian Hyde , fix works.
          Hide
          julianhyde Julian Hyde added a comment -

          Fixed in http://git-wip-us.apache.org/repos/asf/calcite/commit/81b48768.

          Milinda Lakmal Pathirage, The above is the same commit I had previously made on your branch, cherry-picked onto master. Hopefully you'll be able to rebase onto it without too much trouble.

          Show
          julianhyde Julian Hyde added a comment - Fixed in http://git-wip-us.apache.org/repos/asf/calcite/commit/81b48768 . Milinda Lakmal Pathirage , The above is the same commit I had previously made on your branch, cherry-picked onto master. Hopefully you'll be able to rebase onto it without too much trouble.
          Hide
          julianhyde Julian Hyde added a comment -

          Resolved in release 1.6.0 (2016-01-22).

          Show
          julianhyde Julian Hyde added a comment - Resolved in release 1.6.0 (2016-01-22).

            People

            • Assignee:
              julianhyde Julian Hyde
              Reporter:
              julianhyde Julian Hyde
            • Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Development