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

-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.

+1

the patch looks good to me.

Github user asfgit closed the pull request at:

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

committed it

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

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