Uploaded image for project: 'Tajo'
  1. Tajo
  2. TAJO-863

Column order mismatched in the JOIN query with asterisk selection.

    Details

    • Type: Bug
    • Status: Resolved
    • Priority: Minor
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 0.9.0
    • Component/s: None
    • Labels:
      None

      Description

      In the case of the following query, Columns of a nation table should be preceded, but customer is preceded.

      select *
      from nation b
      join customer a on b.n_nationkey = a.c_nationkey
      

      Query's result is

      c_custkey,c_name,c_address,c_nationkey,c_phone,c_acctbal,c_mktsegment,c_comment,n_nationkey,n_name,n_regionkey,n_comment
      -------------------------------
      3,Customer#000000003,MG9kdTD2WBHm,1,11-719-748-3364,7498.12,AUTOMOBILE, deposits eat slyly ironic, even instructions. express foxes detect slyly. blithely even accounts abov,1,ARGENTINA,1,al foxes promise slyly according to the regular accounts. bold requests alon
      5,Customer#000000005,KvpyuHCplrB84WgAiGV6sYpZq7Tj,3,13-750-942-6364,794.47,HOUSEHOLD,n accounts will have to unwind. foxes cajole accor,3,CANADA,1,eas hang ironic, silent packages. slyly regular packages are furiously over the tithes. fluffily bold
      4,Customer#000000004,XxVSJsLAGtn,4,14-128-190-5944,2866.83,MACHINERY, requests. final, regular ideas sleep final accou,4,EGYPT,4,y above the carefully unusual theodolites. final dugouts are quickly across the furiously regular d
      

        Activity

        Hide
        githubbot ASF GitHub Bot added a comment -

        GitHub user babokim opened a pull request:

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

        TAJO-863: Column order mismatched in the JOIN query with asterisk selection

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

        $ git pull https://github.com/babokim/tajo TAJO-863

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

        https://github.com/apache/tajo/pull/31.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 #31


        commit 3faa5db3623aa37ca5a56537df5ec8817cab3cc8
        Author: 김형준 <babokim@babokim-mbp.server.gruter.com>
        Date: 2014-06-03T10:09:47Z

        TAJO-863: Column order mismatched in the JOIN query with asterisk selection.


        Show
        githubbot ASF GitHub Bot added a comment - GitHub user babokim opened a pull request: https://github.com/apache/tajo/pull/31 TAJO-863 : Column order mismatched in the JOIN query with asterisk selection You can merge this pull request into a Git repository by running: $ git pull https://github.com/babokim/tajo TAJO-863 Alternatively you can review and apply these changes as the patch at: https://github.com/apache/tajo/pull/31.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 #31 commit 3faa5db3623aa37ca5a56537df5ec8817cab3cc8 Author: 김형준 <babokim@babokim-mbp.server.gruter.com> Date: 2014-06-03T10:09:47Z TAJO-863 : Column order mismatched in the JOIN query with asterisk selection.
        Hide
        githubbot ASF GitHub Bot added a comment -

        Github user hyunsik commented on the pull request:

        https://github.com/apache/tajo/pull/31#issuecomment-45276274

        +1

        The patch looks good to me. The fix looks straightforward and the unit test is enough.

        Show
        githubbot ASF GitHub Bot added a comment - Github user hyunsik commented on the pull request: https://github.com/apache/tajo/pull/31#issuecomment-45276274 +1 The patch looks good to me. The fix looks straightforward and the unit test is enough.
        Hide
        githubbot ASF GitHub Bot added a comment -

        Github user asfgit closed the pull request at:

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

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

        committed it to master branch. Thank you for your contribution!

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

        SUCCESS: Integrated in Tajo-master-build #236 (See https://builds.apache.org/job/Tajo-master-build/236/)
        TAJO-863: Column order mismatched in the JOIN query with asterisk selection. (Hyoungjun Kim via hyunsik) (hyunsik: rev 47f6f56c7ba6feb20bc991cafdafd160ca53684d)

        • tajo-core/src/test/java/org/apache/tajo/engine/query/TestJoinQuery.java
        • tajo-core/src/test/resources/results/TestJoinQuery/testJoinAsterisk.result
        • tajo-core/src/test/resources/queries/TestJoinQuery/testJoinAsterisk.sql
        • tajo-core/src/main/java/org/apache/tajo/engine/planner/LogicalPlan.java
        • tajo-core/src/test/java/org/apache/tajo/engine/planner/global/TestBroadcastJoinPlan.java
        • CHANGES
        Show
        hudson Hudson added a comment - SUCCESS: Integrated in Tajo-master-build #236 (See https://builds.apache.org/job/Tajo-master-build/236/ ) TAJO-863 : Column order mismatched in the JOIN query with asterisk selection. (Hyoungjun Kim via hyunsik) (hyunsik: rev 47f6f56c7ba6feb20bc991cafdafd160ca53684d) tajo-core/src/test/java/org/apache/tajo/engine/query/TestJoinQuery.java tajo-core/src/test/resources/results/TestJoinQuery/testJoinAsterisk.result tajo-core/src/test/resources/queries/TestJoinQuery/testJoinAsterisk.sql tajo-core/src/main/java/org/apache/tajo/engine/planner/LogicalPlan.java tajo-core/src/test/java/org/apache/tajo/engine/planner/global/TestBroadcastJoinPlan.java CHANGES

          People

          • Assignee:
            hjkim Hyoungjun Kim
            Reporter:
            hjkim Hyoungjun Kim
          • Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development