Details

    • Type: Sub-task
    • Status: Resolved
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 0.11.0
    • Component/s: None
    • Labels:
      None

      Description

      Currently, Tajo handles both UNION and UNION ALL as UNION ALL.
      For UNION, however, result should not have duplicated rows.
      It needs to handle them differently.

        Activity

        Hide
        sirpkt Keuntae Park added a comment - - edited

        Upon the definition of UNION and UNION ALL,
        UNION is equivalent to applying DISTINCT to the result of UNION ALL.
        So, I'm going to handle UNION by rewriting as UNION ALL + GROUP BY.
        Hive also implements UNION by rewriting it to UNION ALL followed by GROUP BY in HIVE-9039.

        Show
        sirpkt Keuntae Park added a comment - - edited Upon the definition of UNION and UNION ALL, UNION is equivalent to applying DISTINCT to the result of UNION ALL. So, I'm going to handle UNION by rewriting as UNION ALL + GROUP BY. Hive also implements UNION by rewriting it to UNION ALL followed by GROUP BY in HIVE-9039 .
        Hide
        githubbot ASF GitHub Bot added a comment -

        GitHub user sirpkt opened a pull request:

        https://github.com/apache/tajo/pull/505

        TAJO-1514: Distinguish UNION and UNION ALL

        This patch distinguishes UNION and UNION ALL, and implements UNION as UNION ALL + GROUP BY

        UNION logical node with distinct flag is written as
        Projection + Group by + Table Subquery + Union

        It also fix test queries wrongly use union like union all

        'mvn clean install' passed in my laptop

        You can merge this pull request into a Git repository by running:

        $ git pull https://github.com/sirpkt/tajo TAJO-1514

        Alternatively you can review and apply these changes as the patch at:

        https://github.com/apache/tajo/pull/505.patch

        To close this pull request, make a commit to your master/trunk branch
        with (at least) the following in the commit message:

        This closes #505


        commit 337919d03c1b7b152b38afdaf9aa799f6440adb2
        Author: Keuntae Park <sirpkt@apache.org>
        Date: 2015-04-03T07:39:41Z

        This patch distinguishes UNION and UNION ALL, and implements UNION as UNION ALL + GROUP BY

        UNION logical node with distinct flag is written as
        Projection + Group by + Table Subquery + Union

        It also fix test queries wrongly use union like union all

        'mvn clean install' passed in my laptop


        Show
        githubbot ASF GitHub Bot added a comment - GitHub user sirpkt opened a pull request: https://github.com/apache/tajo/pull/505 TAJO-1514 : Distinguish UNION and UNION ALL This patch distinguishes UNION and UNION ALL, and implements UNION as UNION ALL + GROUP BY UNION logical node with distinct flag is written as Projection + Group by + Table Subquery + Union It also fix test queries wrongly use union like union all 'mvn clean install' passed in my laptop You can merge this pull request into a Git repository by running: $ git pull https://github.com/sirpkt/tajo TAJO-1514 Alternatively you can review and apply these changes as the patch at: https://github.com/apache/tajo/pull/505.patch To close this pull request, make a commit to your master/trunk branch with (at least) the following in the commit message: This closes #505 commit 337919d03c1b7b152b38afdaf9aa799f6440adb2 Author: Keuntae Park <sirpkt@apache.org> Date: 2015-04-03T07:39:41Z This patch distinguishes UNION and UNION ALL, and implements UNION as UNION ALL + GROUP BY UNION logical node with distinct flag is written as Projection + Group by + Table Subquery + Union It also fix test queries wrongly use union like union all 'mvn clean install' passed in my laptop
        Hide
        githubbot ASF GitHub Bot added a comment -

        Github user hyunsik commented on the pull request:

        https://github.com/apache/tajo/pull/505#issuecomment-89479419

        Unfortunately, I have enough time to review this patch. I'm sorry. I roughly looked over your approach. It looks nice to me.

        Show
        githubbot ASF GitHub Bot added a comment - Github user hyunsik commented on the pull request: https://github.com/apache/tajo/pull/505#issuecomment-89479419 Unfortunately, I have enough time to review this patch. I'm sorry. I roughly looked over your approach. It looks nice to me.
        Hide
        githubbot ASF GitHub Bot added a comment -

        Github user sirpkt commented on the pull request:

        https://github.com/apache/tajo/pull/505#issuecomment-89949099

        Thank you for the comment, @hyunsik !
        I always appreciate your devoted effort on our community,
        so please don't feel sorry about that

        Show
        githubbot ASF GitHub Bot added a comment - Github user sirpkt commented on the pull request: https://github.com/apache/tajo/pull/505#issuecomment-89949099 Thank you for the comment, @hyunsik ! I always appreciate your devoted effort on our community, so please don't feel sorry about that
        Hide
        githubbot ASF GitHub Bot added a comment -

        Github user sirpkt commented on the pull request:

        https://github.com/apache/tajo/pull/505#issuecomment-93638720

        rebased.

        Show
        githubbot ASF GitHub Bot added a comment - Github user sirpkt commented on the pull request: https://github.com/apache/tajo/pull/505#issuecomment-93638720 rebased.
        Hide
        githubbot ASF GitHub Bot added a comment -

        Github user hyunsik commented on the pull request:

        https://github.com/apache/tajo/pull/505#issuecomment-121164355

        Hi @sirpkt,

        Thank you for your work. I'm going to continue this work. I'm sorry for late review.

        Show
        githubbot ASF GitHub Bot added a comment - Github user hyunsik commented on the pull request: https://github.com/apache/tajo/pull/505#issuecomment-121164355 Hi @sirpkt, Thank you for your work. I'm going to continue this work. I'm sorry for late review.
        Hide
        githubbot ASF GitHub Bot added a comment -

        Github user hyunsik commented on the pull request:

        https://github.com/apache/tajo/pull/505#issuecomment-121179551

        Even through it has very trivial conflict against the latest master branch, I can easily merge it to the latest master branch. The patch looks great to me. Here is my +1. I'll commit it shortly after I fix some trivial conflict. Thank you for your contribution.

        Show
        githubbot ASF GitHub Bot added a comment - Github user hyunsik commented on the pull request: https://github.com/apache/tajo/pull/505#issuecomment-121179551 Even through it has very trivial conflict against the latest master branch, I can easily merge it to the latest master branch. The patch looks great to me. Here is my +1. I'll commit it shortly after I fix some trivial conflict. Thank you for your contribution.
        Hide
        githubbot ASF GitHub Bot added a comment -

        Github user jihoonson commented on the pull request:

        https://github.com/apache/tajo/pull/505#issuecomment-121232339

        +1 LGTM!

        Show
        githubbot ASF GitHub Bot added a comment - Github user jihoonson commented on the pull request: https://github.com/apache/tajo/pull/505#issuecomment-121232339 +1 LGTM!
        Hide
        githubbot ASF GitHub Bot added a comment -

        Github user asfgit closed the pull request at:

        https://github.com/apache/tajo/pull/505

        Show
        githubbot ASF GitHub Bot added a comment - Github user asfgit closed the pull request at: https://github.com/apache/tajo/pull/505
        Hide
        hyunsik Hyunsik Choi added a comment -

        I just committed it to master branch. Thank you!

        Show
        hyunsik Hyunsik Choi added a comment - I just committed it to master branch. Thank you!
        Hide
        hudson Hudson added a comment -

        ABORTED: Integrated in Tajo-master-CODEGEN-build #389 (See https://builds.apache.org/job/Tajo-master-CODEGEN-build/389/)
        TAJO-1514: Distinguish UNION and UNION ALL. (contributed by Keuntae Park, committed by hyunsik) (hyunsik: rev 92e3bc1b899036084bba29abe2ea82587cb76c12)

        • tajo-core/src/test/resources/queries/TestUnionQuery/testUnion15.sql
        • tajo-core/src/test/resources/results/TestUnionQuery/testUnionAllWithDifferentAlias.result
        • tajo-plan/src/main/java/org/apache/tajo/plan/LogicalPlanner.java
        • tajo-core/src/test/resources/results/TestUnionQuery/testUnionAll13.result
        • tajo-core/src/test/resources/results/TestUnionQuery/testUnionAll2.result
        • tajo-core/src/test/resources/queries/TestUnionQuery/testUnionAll7.sql
        • tajo-core/src/test/resources/queries/TestUnionQuery/testUnionWithDifferentAlias.sql
        • tajo-core/src/test/resources/queries/TestUnionQuery/testUnionAll3.sql
        • tajo-core/src/test/resources/queries/TestUnionQuery/testUnion14.sql
        • tajo-plan/src/main/java/org/apache/tajo/plan/LogicalPlanPreprocessor.java
        • tajo-core/src/test/resources/results/TestUnionQuery/testUnionAll16.result
        • tajo-core/src/test/resources/results/TestUnionQuery/testUnionAll5.result
        • tajo-core/src/test/resources/queries/TestJoinQuery/testComplexJoinCondition6.sql
        • tajo-core/src/test/resources/results/TestUnionQuery/testUnionWithSameAliasNames.result
        • tajo-core/src/test/resources/queries/TestUnionQuery/testUnion9.sql
        • tajo-core/src/test/resources/queries/TestUnionQuery/testUnionAll16.sql
        • tajo-core/src/test/resources/queries/TestUnionQuery/testUnion11.sql
        • tajo-core/src/test/resources/results/TestUnionQuery/testUnionAll10.result
        • tajo-core/src/test/resources/queries/TestUnionQuery/testUnionAll6.sql
        • tajo-core/src/test/resources/queries/TestJoinQuery/testLeftOuterJoinWithEmptyTable4.sql
        • tajo-core/src/test/resources/queries/TestUnionQuery/testUnionAll11.sql
        • tajo-core/src/test/resources/results/TestUnionQuery/testUnionAll14.result
        • tajo-core/src/test/resources/queries/TestUnionQuery/testUnionAll2.sql
        • tajo-plan/src/main/java/org/apache/tajo/plan/LogicalPlan.java
        • tajo-core/src/test/resources/results/TestUnionQuery/testUnionAll1.result
        • tajo-core/src/test/resources/queries/TestUnionQuery/testUnionAllWithDifferentAlias.sql
        • tajo-core/src/test/resources/results/TestUnionQuery/testUnion1.result
        • tajo-core/src/test/resources/results/TestUnionQuery/testUnionAllWithSameAliasNames.result
        • tajo-core/src/test/resources/results/TestUnionQuery/testUnionAll11.result
        • tajo-core/src/test/resources/queries/TestUnionQuery/testUnion6.sql
        • tajo-core/src/test/resources/queries/TestGroupByQuery/testDistinctAggregationWithUnion1.sql
        • CHANGES
        • tajo-core/src/test/resources/queries/TestUnionQuery/testUnion8.sql
        • tajo-core/src/test/resources/results/TestUnionQuery/testUnion2.result
        • tajo-core/src/test/resources/results/TestUnionQuery/testUnion10.result
        • tajo-core/src/test/resources/queries/TestUnionQuery/testUnion13.sql
        • tajo-core/src/test/resources/queries/TestUnionQuery/testUnionAll13.sql
        • tajo-core/src/test/resources/queries/TestUnionQuery/testUnion16.sql
        • tajo-core/src/test/resources/queries/TestJoinQuery/testComplexJoinCondition7.sql
        • tajo-core/src/test/resources/queries/TestUnionQuery/testUnion12.sql
        • tajo-core/src/test/resources/queries/TestUnionQuery/testUnionAll4.sql
        • tajo-core/src/test/resources/results/TestUnionQuery/testUnionAllWithDifferentAliasAndFunction.result
        • tajo-core/src/test/resources/results/TestUnionQuery/testUnionAll8.result
        • tajo-plan/src/main/java/org/apache/tajo/plan/logical/UnionNode.java
        • tajo-core/src/test/resources/results/TestUnionQuery/testUnionAll15.result
        • tajo-core/src/test/resources/results/TestUnionQuery/testUnionAll12.result
        • tajo-core/src/test/resources/results/TestUnionQuery/testUnionAll4.result
        • tajo-core/src/test/resources/results/TestUnionQuery/testUnionAll7.result
        • tajo-core/src/test/resources/queries/TestUnionQuery/testUnion10.sql
        • tajo-core/src/test/java/org/apache/tajo/engine/query/TestUnionQuery.java
        • tajo-core/src/test/java/org/apache/tajo/engine/planner/TestLogicalPlanner.java
        • tajo-core/src/test/resources/queries/TestUnionQuery/testUnionAll9.sql
        • tajo-core/src/test/resources/queries/TestUnionQuery/testUnionAllWithSameAliasNames.sql
        • tajo-core/src/test/resources/results/TestUnionQuery/testUnion6.result
        • tajo-core/src/test/resources/results/TestUnionQuery/testUnionAll9.result
        • tajo-core/src/test/resources/results/TestUnionQuery/testUnionAll6.result
        • tajo-core/src/test/resources/queries/TestUnionQuery/testUnionAll12.sql
        • tajo-core/src/test/resources/results/TestUnionQuery/testUnionAll3.result
        • tajo-core/src/test/resources/queries/TestUnionQuery/testUnionAll14.sql
        • tajo-core/src/test/resources/queries/TestUnionQuery/testUnionWithDifferentAliasAndFunction.sql
        • tajo-plan/src/main/java/org/apache/tajo/plan/logical/SetOperationNode.java
        • tajo-core/src/test/resources/queries/TestTablePartitions/case2.sql
        • tajo-core/src/test/resources/results/TestUnionQuery/testUnion7.result
        • tajo-core/src/test/resources/queries/TestUnionQuery/testUnionAll5.sql
        • tajo-core/src/test/resources/queries/TestUnionQuery/testUnionAllWithDifferentAliasAndFunction.sql
        • tajo-core/src/test/resources/queries/TestUnionQuery/testUnionAll1.sql
        • tajo-core/src/test/resources/queries/TestUnionQuery/testUnionAll15.sql
        • tajo-core/src/test/resources/queries/TestUnionQuery/testUnionAll10.sql
        • tajo-core/src/test/resources/queries/TestUnionQuery/testUnionAll8.sql
        Show
        hudson Hudson added a comment - ABORTED: Integrated in Tajo-master-CODEGEN-build #389 (See https://builds.apache.org/job/Tajo-master-CODEGEN-build/389/ ) TAJO-1514 : Distinguish UNION and UNION ALL. (contributed by Keuntae Park, committed by hyunsik) (hyunsik: rev 92e3bc1b899036084bba29abe2ea82587cb76c12) tajo-core/src/test/resources/queries/TestUnionQuery/testUnion15.sql tajo-core/src/test/resources/results/TestUnionQuery/testUnionAllWithDifferentAlias.result tajo-plan/src/main/java/org/apache/tajo/plan/LogicalPlanner.java tajo-core/src/test/resources/results/TestUnionQuery/testUnionAll13.result tajo-core/src/test/resources/results/TestUnionQuery/testUnionAll2.result tajo-core/src/test/resources/queries/TestUnionQuery/testUnionAll7.sql tajo-core/src/test/resources/queries/TestUnionQuery/testUnionWithDifferentAlias.sql tajo-core/src/test/resources/queries/TestUnionQuery/testUnionAll3.sql tajo-core/src/test/resources/queries/TestUnionQuery/testUnion14.sql tajo-plan/src/main/java/org/apache/tajo/plan/LogicalPlanPreprocessor.java tajo-core/src/test/resources/results/TestUnionQuery/testUnionAll16.result tajo-core/src/test/resources/results/TestUnionQuery/testUnionAll5.result tajo-core/src/test/resources/queries/TestJoinQuery/testComplexJoinCondition6.sql tajo-core/src/test/resources/results/TestUnionQuery/testUnionWithSameAliasNames.result tajo-core/src/test/resources/queries/TestUnionQuery/testUnion9.sql tajo-core/src/test/resources/queries/TestUnionQuery/testUnionAll16.sql tajo-core/src/test/resources/queries/TestUnionQuery/testUnion11.sql tajo-core/src/test/resources/results/TestUnionQuery/testUnionAll10.result tajo-core/src/test/resources/queries/TestUnionQuery/testUnionAll6.sql tajo-core/src/test/resources/queries/TestJoinQuery/testLeftOuterJoinWithEmptyTable4.sql tajo-core/src/test/resources/queries/TestUnionQuery/testUnionAll11.sql tajo-core/src/test/resources/results/TestUnionQuery/testUnionAll14.result tajo-core/src/test/resources/queries/TestUnionQuery/testUnionAll2.sql tajo-plan/src/main/java/org/apache/tajo/plan/LogicalPlan.java tajo-core/src/test/resources/results/TestUnionQuery/testUnionAll1.result tajo-core/src/test/resources/queries/TestUnionQuery/testUnionAllWithDifferentAlias.sql tajo-core/src/test/resources/results/TestUnionQuery/testUnion1.result tajo-core/src/test/resources/results/TestUnionQuery/testUnionAllWithSameAliasNames.result tajo-core/src/test/resources/results/TestUnionQuery/testUnionAll11.result tajo-core/src/test/resources/queries/TestUnionQuery/testUnion6.sql tajo-core/src/test/resources/queries/TestGroupByQuery/testDistinctAggregationWithUnion1.sql CHANGES tajo-core/src/test/resources/queries/TestUnionQuery/testUnion8.sql tajo-core/src/test/resources/results/TestUnionQuery/testUnion2.result tajo-core/src/test/resources/results/TestUnionQuery/testUnion10.result tajo-core/src/test/resources/queries/TestUnionQuery/testUnion13.sql tajo-core/src/test/resources/queries/TestUnionQuery/testUnionAll13.sql tajo-core/src/test/resources/queries/TestUnionQuery/testUnion16.sql tajo-core/src/test/resources/queries/TestJoinQuery/testComplexJoinCondition7.sql tajo-core/src/test/resources/queries/TestUnionQuery/testUnion12.sql tajo-core/src/test/resources/queries/TestUnionQuery/testUnionAll4.sql tajo-core/src/test/resources/results/TestUnionQuery/testUnionAllWithDifferentAliasAndFunction.result tajo-core/src/test/resources/results/TestUnionQuery/testUnionAll8.result tajo-plan/src/main/java/org/apache/tajo/plan/logical/UnionNode.java tajo-core/src/test/resources/results/TestUnionQuery/testUnionAll15.result tajo-core/src/test/resources/results/TestUnionQuery/testUnionAll12.result tajo-core/src/test/resources/results/TestUnionQuery/testUnionAll4.result tajo-core/src/test/resources/results/TestUnionQuery/testUnionAll7.result tajo-core/src/test/resources/queries/TestUnionQuery/testUnion10.sql tajo-core/src/test/java/org/apache/tajo/engine/query/TestUnionQuery.java tajo-core/src/test/java/org/apache/tajo/engine/planner/TestLogicalPlanner.java tajo-core/src/test/resources/queries/TestUnionQuery/testUnionAll9.sql tajo-core/src/test/resources/queries/TestUnionQuery/testUnionAllWithSameAliasNames.sql tajo-core/src/test/resources/results/TestUnionQuery/testUnion6.result tajo-core/src/test/resources/results/TestUnionQuery/testUnionAll9.result tajo-core/src/test/resources/results/TestUnionQuery/testUnionAll6.result tajo-core/src/test/resources/queries/TestUnionQuery/testUnionAll12.sql tajo-core/src/test/resources/results/TestUnionQuery/testUnionAll3.result tajo-core/src/test/resources/queries/TestUnionQuery/testUnionAll14.sql tajo-core/src/test/resources/queries/TestUnionQuery/testUnionWithDifferentAliasAndFunction.sql tajo-plan/src/main/java/org/apache/tajo/plan/logical/SetOperationNode.java tajo-core/src/test/resources/queries/TestTablePartitions/case2.sql tajo-core/src/test/resources/results/TestUnionQuery/testUnion7.result tajo-core/src/test/resources/queries/TestUnionQuery/testUnionAll5.sql tajo-core/src/test/resources/queries/TestUnionQuery/testUnionAllWithDifferentAliasAndFunction.sql tajo-core/src/test/resources/queries/TestUnionQuery/testUnionAll1.sql tajo-core/src/test/resources/queries/TestUnionQuery/testUnionAll15.sql tajo-core/src/test/resources/queries/TestUnionQuery/testUnionAll10.sql tajo-core/src/test/resources/queries/TestUnionQuery/testUnionAll8.sql
        Hide
        hudson Hudson added a comment -

        SUCCESS: Integrated in Tajo-master-build #749 (See https://builds.apache.org/job/Tajo-master-build/749/)
        TAJO-1514: Distinguish UNION and UNION ALL. (contributed by Keuntae Park, committed by hyunsik) (hyunsik: rev 92e3bc1b899036084bba29abe2ea82587cb76c12)

        • tajo-core/src/test/resources/results/TestUnionQuery/testUnionAllWithSameAliasNames.result
        • tajo-core/src/test/resources/queries/TestUnionQuery/testUnionAll3.sql
        • tajo-plan/src/main/java/org/apache/tajo/plan/LogicalPlanPreprocessor.java
        • tajo-core/src/test/resources/queries/TestUnionQuery/testUnionAll7.sql
        • tajo-core/src/test/resources/results/TestUnionQuery/testUnionWithSameAliasNames.result
        • tajo-core/src/test/resources/results/TestUnionQuery/testUnionAll7.result
        • tajo-core/src/test/resources/queries/TestUnionQuery/testUnion14.sql
        • tajo-core/src/test/resources/results/TestUnionQuery/testUnion2.result
        • tajo-core/src/test/java/org/apache/tajo/engine/planner/TestLogicalPlanner.java
        • tajo-core/src/test/resources/results/TestUnionQuery/testUnionAll14.result
        • tajo-core/src/test/resources/queries/TestUnionQuery/testUnionAll5.sql
        • tajo-core/src/test/resources/results/TestUnionQuery/testUnionAll16.result
        • tajo-core/src/test/resources/results/TestUnionQuery/testUnionAll5.result
        • tajo-core/src/test/resources/queries/TestUnionQuery/testUnionAll11.sql
        • tajo-core/src/test/resources/results/TestUnionQuery/testUnionAll2.result
        • tajo-core/src/test/resources/results/TestUnionQuery/testUnionAll13.result
        • tajo-plan/src/main/java/org/apache/tajo/plan/logical/UnionNode.java
        • tajo-core/src/test/java/org/apache/tajo/engine/query/TestUnionQuery.java
        • tajo-core/src/test/resources/results/TestUnionQuery/testUnionAll10.result
        • tajo-core/src/test/resources/queries/TestUnionQuery/testUnionAll6.sql
        • tajo-core/src/test/resources/results/TestUnionQuery/testUnionAll4.result
        • tajo-core/src/test/resources/results/TestUnionQuery/testUnionAllWithDifferentAlias.result
        • tajo-core/src/test/resources/queries/TestUnionQuery/testUnionAllWithDifferentAliasAndFunction.sql
        • tajo-core/src/test/resources/queries/TestUnionQuery/testUnion6.sql
        • tajo-core/src/test/resources/queries/TestUnionQuery/testUnionAll14.sql
        • tajo-core/src/test/resources/queries/TestUnionQuery/testUnionAll2.sql
        • tajo-core/src/test/resources/queries/TestUnionQuery/testUnion13.sql
        • tajo-core/src/test/resources/queries/TestJoinQuery/testLeftOuterJoinWithEmptyTable4.sql
        • tajo-core/src/test/resources/queries/TestUnionQuery/testUnionWithDifferentAlias.sql
        • tajo-plan/src/main/java/org/apache/tajo/plan/LogicalPlanner.java
        • tajo-core/src/test/resources/queries/TestUnionQuery/testUnionWithDifferentAliasAndFunction.sql
        • tajo-core/src/test/resources/queries/TestTablePartitions/case2.sql
        • tajo-core/src/test/resources/results/TestUnionQuery/testUnion6.result
        • tajo-plan/src/main/java/org/apache/tajo/plan/LogicalPlan.java
        • tajo-core/src/test/resources/queries/TestUnionQuery/testUnion10.sql
        • tajo-core/src/test/resources/queries/TestUnionQuery/testUnionAll12.sql
        • tajo-core/src/test/resources/results/TestUnionQuery/testUnionAll1.result
        • tajo-core/src/test/resources/results/TestUnionQuery/testUnionAll11.result
        • tajo-core/src/test/resources/results/TestUnionQuery/testUnionAll3.result
        • tajo-core/src/test/resources/queries/TestUnionQuery/testUnionAll1.sql
        • tajo-core/src/test/resources/queries/TestUnionQuery/testUnionAll13.sql
        • tajo-core/src/test/resources/queries/TestUnionQuery/testUnion8.sql
        • tajo-core/src/test/resources/results/TestUnionQuery/testUnionAll9.result
        • tajo-core/src/test/resources/queries/TestUnionQuery/testUnion11.sql
        • tajo-plan/src/main/java/org/apache/tajo/plan/logical/SetOperationNode.java
        • tajo-core/src/test/resources/results/TestUnionQuery/testUnionAll12.result
        • tajo-core/src/test/resources/queries/TestUnionQuery/testUnionAll15.sql
        • tajo-core/src/test/resources/results/TestUnionQuery/testUnionAllWithDifferentAliasAndFunction.result
        • tajo-core/src/test/resources/queries/TestUnionQuery/testUnion12.sql
        • tajo-core/src/test/resources/queries/TestUnionQuery/testUnion9.sql
        • tajo-core/src/test/resources/results/TestUnionQuery/testUnionAll15.result
        • tajo-core/src/test/resources/queries/TestUnionQuery/testUnionAll4.sql
        • tajo-core/src/test/resources/results/TestUnionQuery/testUnionAll8.result
        • tajo-core/src/test/resources/queries/TestUnionQuery/testUnion16.sql
        • tajo-core/src/test/resources/queries/TestUnionQuery/testUnionAll9.sql
        • CHANGES
        • tajo-core/src/test/resources/queries/TestUnionQuery/testUnionAll10.sql
        • tajo-core/src/test/resources/results/TestUnionQuery/testUnion10.result
        • tajo-core/src/test/resources/queries/TestJoinQuery/testComplexJoinCondition7.sql
        • tajo-core/src/test/resources/results/TestUnionQuery/testUnion1.result
        • tajo-core/src/test/resources/queries/TestUnionQuery/testUnionAll8.sql
        • tajo-core/src/test/resources/queries/TestJoinQuery/testComplexJoinCondition6.sql
        • tajo-core/src/test/resources/queries/TestGroupByQuery/testDistinctAggregationWithUnion1.sql
        • tajo-core/src/test/resources/queries/TestUnionQuery/testUnion15.sql
        • tajo-core/src/test/resources/results/TestUnionQuery/testUnion7.result
        • tajo-core/src/test/resources/results/TestUnionQuery/testUnionAll6.result
        • tajo-core/src/test/resources/queries/TestUnionQuery/testUnionAllWithDifferentAlias.sql
        • tajo-core/src/test/resources/queries/TestUnionQuery/testUnionAllWithSameAliasNames.sql
        • tajo-core/src/test/resources/queries/TestUnionQuery/testUnionAll16.sql
        Show
        hudson Hudson added a comment - SUCCESS: Integrated in Tajo-master-build #749 (See https://builds.apache.org/job/Tajo-master-build/749/ ) TAJO-1514 : Distinguish UNION and UNION ALL. (contributed by Keuntae Park, committed by hyunsik) (hyunsik: rev 92e3bc1b899036084bba29abe2ea82587cb76c12) tajo-core/src/test/resources/results/TestUnionQuery/testUnionAllWithSameAliasNames.result tajo-core/src/test/resources/queries/TestUnionQuery/testUnionAll3.sql tajo-plan/src/main/java/org/apache/tajo/plan/LogicalPlanPreprocessor.java tajo-core/src/test/resources/queries/TestUnionQuery/testUnionAll7.sql tajo-core/src/test/resources/results/TestUnionQuery/testUnionWithSameAliasNames.result tajo-core/src/test/resources/results/TestUnionQuery/testUnionAll7.result tajo-core/src/test/resources/queries/TestUnionQuery/testUnion14.sql tajo-core/src/test/resources/results/TestUnionQuery/testUnion2.result tajo-core/src/test/java/org/apache/tajo/engine/planner/TestLogicalPlanner.java tajo-core/src/test/resources/results/TestUnionQuery/testUnionAll14.result tajo-core/src/test/resources/queries/TestUnionQuery/testUnionAll5.sql tajo-core/src/test/resources/results/TestUnionQuery/testUnionAll16.result tajo-core/src/test/resources/results/TestUnionQuery/testUnionAll5.result tajo-core/src/test/resources/queries/TestUnionQuery/testUnionAll11.sql tajo-core/src/test/resources/results/TestUnionQuery/testUnionAll2.result tajo-core/src/test/resources/results/TestUnionQuery/testUnionAll13.result tajo-plan/src/main/java/org/apache/tajo/plan/logical/UnionNode.java tajo-core/src/test/java/org/apache/tajo/engine/query/TestUnionQuery.java tajo-core/src/test/resources/results/TestUnionQuery/testUnionAll10.result tajo-core/src/test/resources/queries/TestUnionQuery/testUnionAll6.sql tajo-core/src/test/resources/results/TestUnionQuery/testUnionAll4.result tajo-core/src/test/resources/results/TestUnionQuery/testUnionAllWithDifferentAlias.result tajo-core/src/test/resources/queries/TestUnionQuery/testUnionAllWithDifferentAliasAndFunction.sql tajo-core/src/test/resources/queries/TestUnionQuery/testUnion6.sql tajo-core/src/test/resources/queries/TestUnionQuery/testUnionAll14.sql tajo-core/src/test/resources/queries/TestUnionQuery/testUnionAll2.sql tajo-core/src/test/resources/queries/TestUnionQuery/testUnion13.sql tajo-core/src/test/resources/queries/TestJoinQuery/testLeftOuterJoinWithEmptyTable4.sql tajo-core/src/test/resources/queries/TestUnionQuery/testUnionWithDifferentAlias.sql tajo-plan/src/main/java/org/apache/tajo/plan/LogicalPlanner.java tajo-core/src/test/resources/queries/TestUnionQuery/testUnionWithDifferentAliasAndFunction.sql tajo-core/src/test/resources/queries/TestTablePartitions/case2.sql tajo-core/src/test/resources/results/TestUnionQuery/testUnion6.result tajo-plan/src/main/java/org/apache/tajo/plan/LogicalPlan.java tajo-core/src/test/resources/queries/TestUnionQuery/testUnion10.sql tajo-core/src/test/resources/queries/TestUnionQuery/testUnionAll12.sql tajo-core/src/test/resources/results/TestUnionQuery/testUnionAll1.result tajo-core/src/test/resources/results/TestUnionQuery/testUnionAll11.result tajo-core/src/test/resources/results/TestUnionQuery/testUnionAll3.result tajo-core/src/test/resources/queries/TestUnionQuery/testUnionAll1.sql tajo-core/src/test/resources/queries/TestUnionQuery/testUnionAll13.sql tajo-core/src/test/resources/queries/TestUnionQuery/testUnion8.sql tajo-core/src/test/resources/results/TestUnionQuery/testUnionAll9.result tajo-core/src/test/resources/queries/TestUnionQuery/testUnion11.sql tajo-plan/src/main/java/org/apache/tajo/plan/logical/SetOperationNode.java tajo-core/src/test/resources/results/TestUnionQuery/testUnionAll12.result tajo-core/src/test/resources/queries/TestUnionQuery/testUnionAll15.sql tajo-core/src/test/resources/results/TestUnionQuery/testUnionAllWithDifferentAliasAndFunction.result tajo-core/src/test/resources/queries/TestUnionQuery/testUnion12.sql tajo-core/src/test/resources/queries/TestUnionQuery/testUnion9.sql tajo-core/src/test/resources/results/TestUnionQuery/testUnionAll15.result tajo-core/src/test/resources/queries/TestUnionQuery/testUnionAll4.sql tajo-core/src/test/resources/results/TestUnionQuery/testUnionAll8.result tajo-core/src/test/resources/queries/TestUnionQuery/testUnion16.sql tajo-core/src/test/resources/queries/TestUnionQuery/testUnionAll9.sql CHANGES tajo-core/src/test/resources/queries/TestUnionQuery/testUnionAll10.sql tajo-core/src/test/resources/results/TestUnionQuery/testUnion10.result tajo-core/src/test/resources/queries/TestJoinQuery/testComplexJoinCondition7.sql tajo-core/src/test/resources/results/TestUnionQuery/testUnion1.result tajo-core/src/test/resources/queries/TestUnionQuery/testUnionAll8.sql tajo-core/src/test/resources/queries/TestJoinQuery/testComplexJoinCondition6.sql tajo-core/src/test/resources/queries/TestGroupByQuery/testDistinctAggregationWithUnion1.sql tajo-core/src/test/resources/queries/TestUnionQuery/testUnion15.sql tajo-core/src/test/resources/results/TestUnionQuery/testUnion7.result tajo-core/src/test/resources/results/TestUnionQuery/testUnionAll6.result tajo-core/src/test/resources/queries/TestUnionQuery/testUnionAllWithDifferentAlias.sql tajo-core/src/test/resources/queries/TestUnionQuery/testUnionAllWithSameAliasNames.sql tajo-core/src/test/resources/queries/TestUnionQuery/testUnionAll16.sql

          People

          • Assignee:
            sirpkt Keuntae Park
            Reporter:
            sirpkt Keuntae Park
          • Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development