[HADOOP-8523] test-patch.sh doesn't validate patches before building

• Type: Improvement
• Status: Resolved
• Priority: Minor
• Resolution: Fixed
• Affects Version/s: 0.23.3, 2.0.0-alpha
• Fix Version/s: 3.0.0-alpha1
• Component/s: build
• Labels: newbie
• Target Version/s: 0.23.3
• Assignee: Jack Dintruff
• Reporter: Jack Dintruff
• Created: 21/Jun/12 15:26
• Updated: 12/May/16 18:21
• Resolved: 06/Jul/12 20:32

Description

When running test-patch.sh with an invalid patch (not formatted properly) or one that doesn't compile, the script spends a lot of time building Hadoop before checking to see if the patch is invalid. It would help devs if it checked first just in case we run test-patch.sh with a bad patch file.

Issue Links

• breaks: HADOOP-8584 test-patch.sh should not immediately exit when no tests are added or modified