[MAPREDUCE-1108] Modify test-patch.sh to verify that the number of findBugs warning is always zero

• Type: Bug
• Status: Resolved
• Priority: Major
• Resolution: Implemented
• Component/s: build
• Assignee: Unassigned
• Reporter: Vinod Kumar Vavilapalli
• Created: 14/Oct/09 06:44
• Updated: 25/Apr/15 23:48
• Resolved: 25/Apr/15 23:48

Description

HADOOP-5661 and MAPREDUCE-769 involve going through pains to make sure findBugs warnings become zero. All that effort would be a waste if patches keep ignoring these warnings.

We should modify the Hudson test-patch.sh script to always verify that the findBugs warnings are zero in number. It should scream when the warnings go above zero level and make sure trunk is always at zero findBugs warnings.

And to help contributors about what to do when an un-ignorable warning is spelt out by test-patch.sh, we should modify the output of test-patch.sh to point to src/test/findbugsExcludeFile.xml for suppressing any warnings.

Issue Links

• relates to: HADOOP-6430 Need indication if FindBugs' warnings level isn't 0