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

        Hide
        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
        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
        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
        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
        jihoonson Jihoon Son added a comment -

        Committed to master

        Show
        jihoonson Jihoon Son added a comment - Committed to master
        Hide
        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
        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