[
https://issues.apache.org/jira/browse/HBASE-14310?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14714451#comment-14714451
]
Stephen Yuan Jiang commented on HBASE-14310:
--------------------------------------------
+1 - LGTM
> test-patch.sh should handle spurious non-zero exit code from maven
> ------------------------------------------------------------------
>
> Key: HBASE-14310
> URL: https://issues.apache.org/jira/browse/HBASE-14310
> Project: HBase
> Issue Type: Test
> Reporter: Ted Yu
> Assignee: Ted Yu
> Priority: Minor
> Attachments: 14310-v1.txt
>
>
> Starting last weekend, I saw patch testing abort due to spurious non-zero
> exit code from maven.
> Here are recent examples.
> https://builds.apache.org/job/PreCommit-HBASE-Build/15251/console :
> {quote}
> HBASE-14286 patch is being downloaded at Tue Aug 25 18:49:17 UTC 2015 from
> http://issues.apache.org/jira/secure/attachment/12751767/HBASE-14286.1.patch
> ...
> /home/jenkins/tools/maven/latest/bin/mvn clean package
> checkstyle:checkstyle-aggregate findbugs:findbugs -DskipTests
> -DHBasePatchProcess >
> /home/jenkins/jenkins-slave/workspace/PreCommit-HBASE-Build/patchprocess/trunkJavacWarnings.txt
> 2>&1
> Trunk compilation is broken?
> \{code\}\{code\}
> {quote}
> https://builds.apache.org/job/PreCommit-HBASE-Build/15250/console :
> {quote}
> HBASE-14268 patch is being downloaded at Tue Aug 25 18:19:25 UTC 2015 from
> http://issues.apache.org/jira/secure/attachment/12752280/14268-V5.patch
> ...
> /home/jenkins/tools/maven/latest/bin/mvn clean package
> checkstyle:checkstyle-aggregate findbugs:findbugs -DskipTests
> -DHBasePatchProcess >
> /home/jenkins/jenkins-slave/workspace/PreCommit-HBASE-Build/patchprocess/trunkJavacWarnings.txt
> 2>&1
> Trunk compilation is broken?
> \{code\}\{code\}
> {quote}
> The search in mvn output for 'Compilation failure' returned nothing.
> I verified locally that with 14268-V5.patch, master branch compiled.
> test-patch.sh should handle the spurious exit code so that patches can be
> tested.
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
