hadoop-common-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Allen Wittenauer (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (HADOOP-12002) test-patch.sh needs to verify all of the findbugs tools exist
Date Thu, 28 May 2015 17:28:20 GMT

    [ https://issues.apache.org/jira/browse/HADOOP-12002?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14563295#comment-14563295
] 

Allen Wittenauer commented on HADOOP-12002:
-------------------------------------------

bq. Without -f condition, the expression returns true also if the path is a directory, not
only a real executable. 

I always forget that because as you said:

bq.  it is very rare case.

Frankly, I would have put this entire JIRA as a rare case. ( I mean, why would you only install
only one of the findbugs binaries when we are clearly requesting the findbugs home and not
the location of the findbugs exec???)  So I guess leave it in since people get confused.

> test-patch.sh needs to verify all of the findbugs tools exist
> -------------------------------------------------------------
>
>                 Key: HADOOP-12002
>                 URL: https://issues.apache.org/jira/browse/HADOOP-12002
>             Project: Hadoop Common
>          Issue Type: Bug
>          Components: build
>            Reporter: Sidharta Seethana
>            Assignee: Kengo Seki
>            Priority: Critical
>              Labels: test-patch
>         Attachments: HADOOP-12002.001.patch, HADOOP-12002.002.patch
>
>
> {{test-patch.sh}} was used with {{FINDBUGS_HOME}} set. See below for an example - there
were 4 findbugs warnings generated - however, {{test-patch.sh}} doesn't seem to realize that
there are missing findbugs tools and +1s the finbugs check. 
> {quote}
>  Running findbugs in hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-nodemanager
> mvn clean test findbugs:findbugs -DskipTests -DhadoopPatchProcess > /private/tmp/hadoop-test-patch/71089/patchFindBugsOutputhadoop-yarn-server-nodemanager.txt
2>&1
> <snip>hadoop/dev-support/test-patch.sh: line 1907: /usr/local/Cellar/findbugs/3.0.0/bin/setBugDatabaseInfo:
No such file or directory
> <snip>hadoop/dev-support/test-patch.sh: line 1915: /usr/local/Cellar/findbugs/3.0.0/bin/filterBugs:
No such file or directory
> Found  Findbugs warnings (<snip>hadoop/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-nodemanager/target/findbugsXml.xml)
> <snip>hadoop/dev-support/test-patch.sh: line 1921: /usr/local/Cellar/findbugs/3.0.0/bin/convertXmlToText:
No such file or directory
> [Mon May 18 18:08:52 PDT 2015 DEBUG]: Stop clock
> Elapsed time:   0m 38s
> {quote}
> Findbugs check reported as successful : 
> {quote}
> |  +1  |           findbugs  |  0m 38s    | The patch does not introduce any 
> |      |                     |            | new Findbugs (version 3.0.0)
> |      |                     |            | warnings.
> |      |                     |  23m 51s   | 
> {quote}



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Mime
View raw message