hadoop-common-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Sidharta Seethana (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:42:19 GMT

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

Sidharta Seethana commented on HADOOP-12002:
--------------------------------------------

The mistake I made was to set {{FINDBUGS_HOME}} to  {{/usr/local/Cellar/findbugs/3.0.0}} instead
of {{/usr/local/Cellar/findbugs/3.0.0/libexec}} as specified below. So this issue is indeed
not as common as I thought it would be. That said, adding a +1 due to an invalid {{FINDBUGS_HOME}}
could cause unnecessary patch iterations on the corresponding JIRA (as it did in my case).


{code}
brew info findbugs
findbugs: stable 3.0.0
http://findbugs.sourceforge.net/index.html
Conflicts with: fb-client
/usr/local/Cellar/findbugs/3.0.0 (156 files, 11M) *
  Built from source
From: https://github.com/Homebrew/homebrew/blob/master/Library/Formula/findbugs.rb
==> Caveats
Some tools might refer findbugs by env variable.
After installation, set $FINDBUGS_HOME in your profile:
  export FINDBUGS_HOME=/usr/local/Cellar/findbugs/3.0.0/libexec
{brew info findbugs
findbugs: stable 3.0.0
http://findbugs.sourceforge.net/index.html
Conflicts with: fb-client
/usr/local/Cellar/findbugs/3.0.0 (156 files, 11M) *
  Built from source
From: https://github.com/Homebrew/homebrew/blob/master/Library/Formula/findbugs.rb
==> Caveats
Some tools might refer findbugs by env variable.
After installation, set $FINDBUGS_HOME in your profile:
  export FINDBUGS_HOME=/usr/local/Cellar/findbugs/3.0.0/libexec
{code}



> 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