[TAJO-1237] Fix missing maven-module for pullserver - ASF JIRA

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

Options
- Sort By Name
- Sort By Date
- Ascending
- Descending

Attachments

TAJO-1237.patch

09/Dec/14 09:22

2 kB

Jinho Kim

Activity

Descending order - Click to sort in ascending order

Hide

Permalink

Hudson added a comment - 10/Dec/14 02:38

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 added a comment - 10/Dec/14 02:38 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

Permalink

Hudson added a comment - 10/Dec/14 02:37

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 added a comment - 10/Dec/14 02:37 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

Permalink

Jinho Kim added a comment - 10/Dec/14 02:22

committed it

Show

Jinho Kim added a comment - 10/Dec/14 02:22 committed it

Hide

Permalink

ASF GitHub Bot added a comment - 10/Dec/14 02:22

Github user asfgit closed the pull request at:

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

Show

ASF GitHub Bot added a comment - 10/Dec/14 02:22 Github user asfgit closed the pull request at: https://github.com/apache/tajo/pull/293

Hide

Permalink

Hyunsik Choi added a comment - 09/Dec/14 22:03

the patch looks good to me.

Show

Hyunsik Choi added a comment - 09/Dec/14 22:03 +1 the patch looks good to me.

Hide

Permalink

Tajo QA added a comment - 09/Dec/14 09:55

-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

Tajo QA added a comment - 09/Dec/14 09:55 -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

Permalink

ASF GitHub Bot added a comment - 09/Dec/14 06:28

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

ASF GitHub Bot added a comment - 09/Dec/14 06:28 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:

Jinho Kim

Reporter:

Jinho Kim

Votes:

0 Vote for this issue

Watchers:

3 Start watching this issue

Dates

Created:

09/Dec/14 06:25

Updated:

10/Dec/14 02:38

Resolved:

10/Dec/14 02:22

Development

Agile

View on Board