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

Extend JoinUnionTransposeRule to match Union instead of LogicalUnion

    Details

    • Type: Improvement
    • Status: Closed
    • Priority: Minor
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 1.7.0
    • Component/s: core
    • Labels:
      None

      Description

      The JoinUnionTransposeRule is fired if one of the operands is a Union, but inside the onMatch() method, the code checks if one of the operands is an instance of LogicalUnion. External implementations would benefit from changing the rule to match Union instead.

      The dev mailing list discussion can be found here.

        Activity

        Show
        jcamachorodriguez Jesus Camacho Rodriguez added a comment - Fixed in http://git-wip-us.apache.org/repos/asf/calcite/commit/50c4b06 . Thanks Vasia Kalavri !
        Hide
        julianhyde Julian Hyde added a comment -

        Resolved in release 1.7.0 (2016-03-22).

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

          People

          • Assignee:
            julianhyde Julian Hyde
            Reporter:
            vkalavri Vasia Kalavri
          • Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development