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

            is depended upon by

            New Feature - A new feature of the product, which has yet to be developed. CALCITE-968 Support stream joins

            • Major - Major loss of function.
            • Closed

              Activity

              Ascending order - Click to sort in descending order
              Hide
              Permalink
              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
              Permalink
              milinda Milinda Lakmal Pathirage added a comment -

              Julian Hyde, fix works.

              Show
              milinda Milinda Lakmal Pathirage added a comment - Julian Hyde , fix works.
              Hide
              Permalink
              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
              Permalink
              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