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

Simple query doesn't work in Web UI

    Details

    • Type: Bug
    • Status: Resolved
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: 0.10.0
    • Fix Version/s: 0.11.0, 0.10.1
    • Component/s: Web UI
    • Labels:
      None

      Description

      Some query which doesn't use a worker, for example like 'select * from tpch.lineitem', doesn't work.

      It returns no row.

      1. TAJO-1147.patch
        0.8 kB
        Jongyoung Park

        Activity

        Hide
        hudson Hudson added a comment -

        ABORTED: Integrated in Tajo-master-CODEGEN-build #269 (See https://builds.apache.org/job/Tajo-master-CODEGEN-build/269/)
        TAJO-1147: Simple query doesn't work in Web UI. (blrunner: rev a01292f7e42966451ee333cd6d99ed0eaa21e793)

        • CHANGES
        • tajo-core/src/main/java/org/apache/tajo/webapp/QueryExecutorServlet.java
        Show
        hudson Hudson added a comment - ABORTED: Integrated in Tajo-master-CODEGEN-build #269 (See https://builds.apache.org/job/Tajo-master-CODEGEN-build/269/ ) TAJO-1147 : Simple query doesn't work in Web UI. (blrunner: rev a01292f7e42966451ee333cd6d99ed0eaa21e793) CHANGES tajo-core/src/main/java/org/apache/tajo/webapp/QueryExecutorServlet.java
        Hide
        hudson Hudson added a comment -

        ABORTED: Integrated in Tajo-master-build #631 (See https://builds.apache.org/job/Tajo-master-build/631/)
        TAJO-1147: Simple query doesn't work in Web UI. (blrunner: rev a01292f7e42966451ee333cd6d99ed0eaa21e793)

        • tajo-core/src/main/java/org/apache/tajo/webapp/QueryExecutorServlet.java
        • CHANGES
        Show
        hudson Hudson added a comment - ABORTED: Integrated in Tajo-master-build #631 (See https://builds.apache.org/job/Tajo-master-build/631/ ) TAJO-1147 : Simple query doesn't work in Web UI. (blrunner: rev a01292f7e42966451ee333cd6d99ed0eaa21e793) tajo-core/src/main/java/org/apache/tajo/webapp/QueryExecutorServlet.java CHANGES
        Hide
        githubbot ASF GitHub Bot added a comment -

        Github user asfgit closed the pull request at:

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

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

        Github user blrunner commented on the pull request:

        https://github.com/apache/tajo/pull/461#issuecomment-86361330

        +1

        It looks good to me and it runs successfully as expected.

        Show
        githubbot ASF GitHub Bot added a comment - Github user blrunner commented on the pull request: https://github.com/apache/tajo/pull/461#issuecomment-86361330 +1 It looks good to me and it runs successfully as expected.
        Hide
        blrunner Jaehwa Jung added a comment -

        Could you rebase the patch against latest master branch?

        Show
        blrunner Jaehwa Jung added a comment - Could you rebase the patch against latest master branch?
        Hide
        githubbot ASF GitHub Bot added a comment -

        Github user blrunner commented on the pull request:

        https://github.com/apache/tajo/pull/461#issuecomment-85529306

        Thanks for your contribution.
        I'll review the patch tomorrow.

        Show
        githubbot ASF GitHub Bot added a comment - Github user blrunner commented on the pull request: https://github.com/apache/tajo/pull/461#issuecomment-85529306 Thanks for your contribution. I'll review the patch tomorrow.
        Hide
        tajoqa Tajo QA added a comment -

        -1 overall. Here are the results of testing the latest attachment
        http://issues.apache.org/jira/secure/attachment/12706872/TAJO-1147.patch
        against master revision release-0.9.0-rc0-217-g3e9a2dd.

        +1 @author. The patch does not contain any @author tags.

        -1 tests included. The patch doesn't appear to include any new or modified tests.
        Please justify why no new tests are needed for this patch.
        Also please list what manual steps were performed to verify this patch.

        +1 javac. The applied patch does not increase the total number of javac compiler warnings.

        +1 javadoc. The applied patch does not increase the total number of javadoc warnings.

        +1 checkstyle. The patch generated 0 code style errors.

        -1 findbugs. The patch appears to introduce 2 new Findbugs (version 2.0.3) warnings.

        +1 release audit. The applied patch does not increase the total number of release audit warnings.

        -1 core tests. The patch failed these unit tests in tajo-core:
        org.apache.tajo.engine.query.TestSelectQuery

        Test results: https://builds.apache.org/job/PreCommit-TAJO-Build/654//testReport/
        Findbugs warnings: https://builds.apache.org/job/PreCommit-TAJO-Build/654//artifact/incubator-tajo/patchprocess/newPatchFindbugsWarningstajo-core.html
        Console output: https://builds.apache.org/job/PreCommit-TAJO-Build/654//console

        This message is automatically generated.

        Show
        tajoqa Tajo QA added a comment - -1 overall. Here are the results of testing the latest attachment http://issues.apache.org/jira/secure/attachment/12706872/TAJO-1147.patch against master revision release-0.9.0-rc0-217-g3e9a2dd. +1 @author. The patch does not contain any @author tags. -1 tests included. The patch doesn't appear to include any new or modified tests. Please justify why no new tests are needed for this patch. Also please list what manual steps were performed to verify this patch. +1 javac. The applied patch does not increase the total number of javac compiler warnings. +1 javadoc. The applied patch does not increase the total number of javadoc warnings. +1 checkstyle. The patch generated 0 code style errors. -1 findbugs. The patch appears to introduce 2 new Findbugs (version 2.0.3) warnings. +1 release audit. The applied patch does not increase the total number of release audit warnings. -1 core tests. The patch failed these unit tests in tajo-core: org.apache.tajo.engine.query.TestSelectQuery Test results: https://builds.apache.org/job/PreCommit-TAJO-Build/654//testReport/ Findbugs warnings: https://builds.apache.org/job/PreCommit-TAJO-Build/654//artifact/incubator-tajo/patchprocess/newPatchFindbugsWarningstajo-core.html Console output: https://builds.apache.org/job/PreCommit-TAJO-Build/654//console This message is automatically generated.
        Hide
        githubbot ASF GitHub Bot added a comment -

        GitHub user eminency opened a pull request:

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

        TAJO-1147: Simple query doesn't work in Web UI

        'resultRows' can be -1.
        -1 means TajoConstants.UNKNOWN_ROW_NUMBER.

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

        $ git pull https://github.com/eminency/tajo TAJO-1147

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

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


        commit def552d1e1a2872263fa3a9f19f50238e57e22c7
        Author: Jongyoung Park <eminency@gmail.com>
        Date: 2015-03-24T10:05:24Z

        resultRows can be -1(TajoConstants.UNKNOWN_ROW_NUMBER)


        Show
        githubbot ASF GitHub Bot added a comment - GitHub user eminency opened a pull request: https://github.com/apache/tajo/pull/461 TAJO-1147 : Simple query doesn't work in Web UI 'resultRows' can be -1. -1 means TajoConstants.UNKNOWN_ROW_NUMBER. You can merge this pull request into a Git repository by running: $ git pull https://github.com/eminency/tajo TAJO-1147 Alternatively you can review and apply these changes as the patch at: https://github.com/apache/tajo/pull/461.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 #461 commit def552d1e1a2872263fa3a9f19f50238e57e22c7 Author: Jongyoung Park <eminency@gmail.com> Date: 2015-03-24T10:05:24Z resultRows can be -1(TajoConstants.UNKNOWN_ROW_NUMBER)

          People

          • Assignee:
            eminency Jongyoung Park
            Reporter:
            eminency Jongyoung Park
          • Votes:
            0 Vote for this issue
            Watchers:
            4 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development