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

Extend JoinUnionTransposeRule to match Union instead of LogicalUnion

VotersWatch issueWatchersLinkCloneUpdate Comment AuthorReplace String in CommentUpdate Comment VisibilityDelete Comments
    XMLWordPrintableJSON

    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.

        Attachments

          Activity

            People

            • Assignee:
              julianhyde Julian Hyde
              Reporter:
              vkalavri Vasia Kalavri

              Dates

              • Created:
                Updated:
                Resolved:

                Issue deployment