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

Fix missing maven-module for pullserver

    Details

    • Type: Bug
    • Status: Resolved
    • Priority: Critical
    • Resolution: Fixed
    • Affects Version/s: 0.10.0
    • Fix Version/s: 0.10.0
    • Component/s: Build
    • Labels:
      None

      Description

      tajo-yarn-pullserver was changed to tajo-pullserver. but there are some missing codes

        Activity

        Hide
        hudson Hudson added a comment -

        SUCCESS: Integrated in Tajo-master-build #491 (See https://builds.apache.org/job/Tajo-master-build/491/)
        TAJO-1237: Fix missing maven-module for pullserver. (jinho) (jhkim: rev 72256fcff2c77222e278cb690efb8f669e926a29)

        • BUILDING
        • tajo-core/pom.xml
        • tajo-pullserver/pom.xml
        • tajo-project/pom.xml
        • CHANGES
        • tajo-dist/pom.xml
        Show
        hudson Hudson added a comment - SUCCESS: Integrated in Tajo-master-build #491 (See https://builds.apache.org/job/Tajo-master-build/491/ ) TAJO-1237 : Fix missing maven-module for pullserver. (jinho) (jhkim: rev 72256fcff2c77222e278cb690efb8f669e926a29) BUILDING tajo-core/pom.xml tajo-pullserver/pom.xml tajo-project/pom.xml CHANGES tajo-dist/pom.xml
        Hide
        hudson Hudson added a comment -

        FAILURE: Integrated in Tajo-master-CODEGEN-build #132 (See https://builds.apache.org/job/Tajo-master-CODEGEN-build/132/)
        TAJO-1237: Fix missing maven-module for pullserver. (jinho) (jhkim: rev 72256fcff2c77222e278cb690efb8f669e926a29)

        • CHANGES
        • tajo-pullserver/pom.xml
        • tajo-project/pom.xml
        • tajo-dist/pom.xml
        • tajo-core/pom.xml
        • BUILDING
        Show
        hudson Hudson added a comment - FAILURE: Integrated in Tajo-master-CODEGEN-build #132 (See https://builds.apache.org/job/Tajo-master-CODEGEN-build/132/ ) TAJO-1237 : Fix missing maven-module for pullserver. (jinho) (jhkim: rev 72256fcff2c77222e278cb690efb8f669e926a29) CHANGES tajo-pullserver/pom.xml tajo-project/pom.xml tajo-dist/pom.xml tajo-core/pom.xml BUILDING
        Hide
        jhkim Jinho Kim added a comment -

        committed it

        Show
        jhkim Jinho Kim added a comment - committed it
        Hide
        githubbot ASF GitHub Bot added a comment -

        Github user asfgit closed the pull request at:

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

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

        +1

        the patch looks good to me.

        Show
        hyunsik Hyunsik Choi added a comment - +1 the patch looks good to me.
        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/12685979/TAJO-1237.patch
        against master revision release-0.9.0-rc0-79-g661c7e2.

        +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 cause Findbugs (version 2.0.3) to fail.

        -1 release audit. The applied patch generated 210 release audit warnings.

        +1 core tests. The patch passed unit tests in tajo-core tajo-dist tajo-pullserver.

        Test results: https://builds.apache.org/job/PreCommit-TAJO-Build/538//testReport/
        Release audit warnings: https://builds.apache.org/job/PreCommit-TAJO-Build/538//artifact/incubator-tajo/patchprocess/patchReleaseAuditProblems.txt
        Findbugs results: https://builds.apache.org/job/PreCommit-TAJO-Build/538//findbugsResult
        Console output: https://builds.apache.org/job/PreCommit-TAJO-Build/538//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/12685979/TAJO-1237.patch against master revision release-0.9.0-rc0-79-g661c7e2. +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 cause Findbugs (version 2.0.3) to fail. -1 release audit. The applied patch generated 210 release audit warnings. +1 core tests. The patch passed unit tests in tajo-core tajo-dist tajo-pullserver. Test results: https://builds.apache.org/job/PreCommit-TAJO-Build/538//testReport/ Release audit warnings: https://builds.apache.org/job/PreCommit-TAJO-Build/538//artifact/incubator-tajo/patchprocess/patchReleaseAuditProblems.txt Findbugs results: https://builds.apache.org/job/PreCommit-TAJO-Build/538//findbugsResult Console output: https://builds.apache.org/job/PreCommit-TAJO-Build/538//console This message is automatically generated.
        Hide
        githubbot ASF GitHub Bot added a comment -

        GitHub user jinossy opened a pull request:

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

        TAJO-1237: Fix missing maven-module for pullserver

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

        $ git pull https://github.com/jinossy/tajo TAJO-1237

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

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


        commit 2517ecc1b3dcc08cad7eef7da9e379e5b372938a
        Author: jhkim <jhkim@apache.org>
        Date: 2014-12-09T06:27:36Z

        TAJO-1237: Fix missing maven-module for pullserver


        Show
        githubbot ASF GitHub Bot added a comment - GitHub user jinossy opened a pull request: https://github.com/apache/tajo/pull/293 TAJO-1237 : Fix missing maven-module for pullserver You can merge this pull request into a Git repository by running: $ git pull https://github.com/jinossy/tajo TAJO-1237 Alternatively you can review and apply these changes as the patch at: https://github.com/apache/tajo/pull/293.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 #293 commit 2517ecc1b3dcc08cad7eef7da9e379e5b372938a Author: jhkim <jhkim@apache.org> Date: 2014-12-09T06:27:36Z TAJO-1237 : Fix missing maven-module for pullserver

          People

          • Assignee:
            jhkim Jinho Kim
            Reporter:
            jhkim Jinho Kim
          • Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development