1) we should add OK_RELEASEAUDIT_WARNINGS to the properties file..
2) Every project should define these numbers even if the project has zero warnings for findbugs/releaseaudit/javadoc. Thatway we dont have to pre-build the trunk to determine the numbers.
HADOOP-7008.patch patch seem to run findbug on trunk if the OK_FINDBUGS_WARNINGS is not defined or the warnings not equal to zero.
Is there any reason why we are running findbugs if its already defined?
I think all the projects should just define the values for these 3 properties. test-patch.sh should directly apply the patch and determine the warnings and do a +1 or -1 (if the numbers are greater than the numbers defined in the properties file.)