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

NameResolver cannot find nested records properly

        Details

        • Type: Bug
        • Status: Resolved
        • Priority: Major
        • Resolution: Fixed
        • Affects Version/s: None
        • Fix Version/s: 0.11.0
        • Component/s: Planner/Optimizer
        • Labels:
          None

          Description

          See the title.
          You can reproduce as follows:

          default> CREATE EXTERNAL TABLE json_test (
id TEXT, type TEXT, url TEXT,
actor RECORD (
  id bigint,
  login text
),
payload RECORD (
  push_id bigint,
  commits RECORD (
    message text,
    url text
  )
),
public BOOLEAN,
created_at text
)
USING JSON 
LOCATION 'path'
;
default>
default> select type, payload.commits.message as msg from gh_json_test limit 3;
ERROR: column 'payload.commits.message' does not exist

            Activity

            Ascending order - Click to sort in descending order
            Hide
            Permalink
            githubbot ASF GitHub Bot added a comment -

            GitHub user jihoonson opened a pull request:

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

            TAJO-1752: NameResolver cannot find nested records properly

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

            $ git pull https://github.com/jihoonson/tajo-2 TAJO-1752

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

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

            commit e9e72af6cde84a72eac919cfee9bc78b82ae6041
            Author: Jihoon Son <jihoonson@apache.org>
            Date: 2015-08-12T06:40:40Z

            TAJO-1752

            Show
            githubbot ASF GitHub Bot added a comment - GitHub user jihoonson opened a pull request: https://github.com/apache/tajo/pull/686 TAJO-1752 : NameResolver cannot find nested records properly You can merge this pull request into a Git repository by running: $ git pull https://github.com/jihoonson/tajo-2 TAJO-1752 Alternatively you can review and apply these changes as the patch at: https://github.com/apache/tajo/pull/686.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 #686 commit e9e72af6cde84a72eac919cfee9bc78b82ae6041 Author: Jihoon Son <jihoonson@apache.org> Date: 2015-08-12T06:40:40Z TAJO-1752
            Hide
            Permalink
            githubbot ASF GitHub Bot added a comment -

            Github user jinossy commented on the pull request:

            https://github.com/apache/tajo/pull/686#issuecomment-130236958

            +1 LGTM!

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

            Github user jihoonson commented on the pull request:

            https://github.com/apache/tajo/pull/686#issuecomment-130237528

            Thanks for your quick review!

            Show
            githubbot ASF GitHub Bot added a comment - Github user jihoonson commented on the pull request: https://github.com/apache/tajo/pull/686#issuecomment-130237528 Thanks for your quick review!
            Hide
            Permalink
            githubbot ASF GitHub Bot added a comment -

            Github user asfgit closed the pull request at:

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

            Show
            githubbot ASF GitHub Bot added a comment - Github user asfgit closed the pull request at: https://github.com/apache/tajo/pull/686
            Hide
            Permalink
            jihoonson Jihoon Son added a comment -

            Committed to master

            Show
            jihoonson Jihoon Son added a comment - Committed to master
            Hide
            Permalink
            hudson Hudson added a comment -

            FAILURE: Integrated in Tajo-master-CODEGEN-build #435 (See https://builds.apache.org/job/Tajo-master-CODEGEN-build/435/)
            TAJO-1752: NameResolver cannot find nested records properly. (jihoonson: rev fd6a95180ecfa05df7cd0b1a8a957057c9788ad6)

            • tajo-core/src/test/java/org/apache/tajo/engine/query/TestSelectNestedRecord.java
            • CHANGES
            • tajo-common/src/main/java/org/apache/tajo/util/StringUtils.java
            • tajo-core/src/test/resources/queries/TestSelectNestedRecord/testSelect3.sql
            • tajo-core/src/test/resources/results/TestSelectNestedRecord/testSelect3.result
            • tajo-plan/src/main/java/org/apache/tajo/plan/nameresolver/NameResolver.java
            • tajo-core/src/test/resources/queries/TestSelectNestedRecord/sample2_ddl.sql
            • tajo-catalog/tajo-catalog-common/src/main/java/org/apache/tajo/catalog/NestedPathUtil.java
            • tajo-core/src/test/resources/dataset/TestSelectNestedRecord/sample2/sample2.json
            Show
            hudson Hudson added a comment - FAILURE: Integrated in Tajo-master-CODEGEN-build #435 (See https://builds.apache.org/job/Tajo-master-CODEGEN-build/435/ ) TAJO-1752 : NameResolver cannot find nested records properly. (jihoonson: rev fd6a95180ecfa05df7cd0b1a8a957057c9788ad6) tajo-core/src/test/java/org/apache/tajo/engine/query/TestSelectNestedRecord.java CHANGES tajo-common/src/main/java/org/apache/tajo/util/StringUtils.java tajo-core/src/test/resources/queries/TestSelectNestedRecord/testSelect3.sql tajo-core/src/test/resources/results/TestSelectNestedRecord/testSelect3.result tajo-plan/src/main/java/org/apache/tajo/plan/nameresolver/NameResolver.java tajo-core/src/test/resources/queries/TestSelectNestedRecord/sample2_ddl.sql tajo-catalog/tajo-catalog-common/src/main/java/org/apache/tajo/catalog/NestedPathUtil.java tajo-core/src/test/resources/dataset/TestSelectNestedRecord/sample2/sample2.json
            Hide
            Permalink
            hudson Hudson added a comment -

            ABORTED: Integrated in Tajo-master-build #797 (See https://builds.apache.org/job/Tajo-master-build/797/)
            TAJO-1752: NameResolver cannot find nested records properly. (jihoonson: rev fd6a95180ecfa05df7cd0b1a8a957057c9788ad6)

            • tajo-core/src/test/java/org/apache/tajo/engine/query/TestSelectNestedRecord.java
            • tajo-plan/src/main/java/org/apache/tajo/plan/nameresolver/NameResolver.java
            • tajo-core/src/test/resources/queries/TestSelectNestedRecord/testSelect3.sql
            • tajo-catalog/tajo-catalog-common/src/main/java/org/apache/tajo/catalog/NestedPathUtil.java
            • tajo-core/src/test/resources/queries/TestSelectNestedRecord/sample2_ddl.sql
            • tajo-core/src/test/resources/dataset/TestSelectNestedRecord/sample2/sample2.json
            • tajo-core/src/test/resources/results/TestSelectNestedRecord/testSelect3.result
            • tajo-common/src/main/java/org/apache/tajo/util/StringUtils.java
            • CHANGES
            Show
            hudson Hudson added a comment - ABORTED: Integrated in Tajo-master-build #797 (See https://builds.apache.org/job/Tajo-master-build/797/ ) TAJO-1752 : NameResolver cannot find nested records properly. (jihoonson: rev fd6a95180ecfa05df7cd0b1a8a957057c9788ad6) tajo-core/src/test/java/org/apache/tajo/engine/query/TestSelectNestedRecord.java tajo-plan/src/main/java/org/apache/tajo/plan/nameresolver/NameResolver.java tajo-core/src/test/resources/queries/TestSelectNestedRecord/testSelect3.sql tajo-catalog/tajo-catalog-common/src/main/java/org/apache/tajo/catalog/NestedPathUtil.java tajo-core/src/test/resources/queries/TestSelectNestedRecord/sample2_ddl.sql tajo-core/src/test/resources/dataset/TestSelectNestedRecord/sample2/sample2.json tajo-core/src/test/resources/results/TestSelectNestedRecord/testSelect3.result tajo-common/src/main/java/org/apache/tajo/util/StringUtils.java CHANGES

              People

              • Assignee:
                jihoonson Jihoon Son
                Reporter:
                jihoonson Jihoon Son
              • Votes:
                0 Vote for this issue
                Watchers:
                3 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved:

                  Development