+1 overall. Here are the results of testing the latest attachment
http://issues.apache.org/jira/secure/attachment/12546528/HADOOP-8839.patch
against trunk revision .

+1 @author. The patch does not contain any @author tags.

+1 tests included. The patch appears to include 1 new or modified test files.

+1 javac. The applied patch does not increase the total number of javac compiler warnings.

+1 javadoc. The javadoc tool did not generate any warning messages.

+1 eclipse:eclipse. The patch built with eclipse:eclipse.

+1 findbugs. The patch does not introduce any new Findbugs (version 1.3.9) warnings.

+1 release audit. The applied patch does not increase the total number of release audit warnings.

+1 core tests. The patch passed unit tests in .

+1 contrib tests. The patch passed contrib unit tests.

Test results: https://builds.apache.org/job/PreCommit-HADOOP-Build/1517//testReport/
Console output: https://builds.apache.org/job/PreCommit-HADOOP-Build/1517//console

This message is automatically generated.

+1, the patch looks good to me.

Thanks Aaron, committed to trunk.

Integrated in Hadoop-Common-trunk-Commit #2791 (See https://builds.apache.org/job/Hadoop-Common-trunk-Commit/2791/)
HADOOP-8839. test-patch's -1 on @author tag presence doesn't cause a -1 to the overall result. Contributed by Harsh J. (harsh) (Revision 1391884)

Result = SUCCESS
harsh : http://svn.apache.org/viewcvs.cgi/?root=Apache-SVN&view=rev&rev=1391884
Files :

• /hadoop/common/trunk/dev-support/test-patch.sh
• /hadoop/common/trunk/hadoop-common-project/hadoop-common/CHANGES.txt

Integrated in Hadoop-Mapreduce-trunk-Commit #2813 (See https://builds.apache.org/job/Hadoop-Mapreduce-trunk-Commit/2813/)
HADOOP-8839. test-patch's -1 on @author tag presence doesn't cause a -1 to the overall result. Contributed by Harsh J. (harsh) (Revision 1391884)

Result = FAILURE
harsh : http://svn.apache.org/viewcvs.cgi/?root=Apache-SVN&view=rev&rev=1391884
Files :

• /hadoop/common/trunk/dev-support/test-patch.sh
• /hadoop/common/trunk/hadoop-common-project/hadoop-common/CHANGES.txt

Integrated in Hadoop-Hdfs-trunk-Commit #2854 (See https://builds.apache.org/job/Hadoop-Hdfs-trunk-Commit/2854/)
HADOOP-8839. test-patch's -1 on @author tag presence doesn't cause a -1 to the overall result. Contributed by Harsh J. (harsh) (Revision 1391884)

Result = SUCCESS
harsh : http://svn.apache.org/viewcvs.cgi/?root=Apache-SVN&view=rev&rev=1391884
Files :

• /hadoop/common/trunk/dev-support/test-patch.sh
• /hadoop/common/trunk/hadoop-common-project/hadoop-common/CHANGES.txt

Integrated in Hadoop-Hdfs-trunk #1181 (See https://builds.apache.org/job/Hadoop-Hdfs-trunk/1181/)
HADOOP-8839. test-patch's -1 on @author tag presence doesn't cause a -1 to the overall result. Contributed by Harsh J. (harsh) (Revision 1391884)

Result = SUCCESS
harsh : http://svn.apache.org/viewcvs.cgi/?root=Apache-SVN&view=rev&rev=1391884
Files :

• /hadoop/common/trunk/dev-support/test-patch.sh
• /hadoop/common/trunk/hadoop-common-project/hadoop-common/CHANGES.txt

Integrated in Hadoop-Mapreduce-trunk #1212 (See https://builds.apache.org/job/Hadoop-Mapreduce-trunk/1212/)
HADOOP-8839. test-patch's -1 on @author tag presence doesn't cause a -1 to the overall result. Contributed by Harsh J. (harsh) (Revision 1391884)

Result = FAILURE
harsh : http://svn.apache.org/viewcvs.cgi/?root=Apache-SVN&view=rev&rev=1391884
Files :

• /hadoop/common/trunk/dev-support/test-patch.sh
• /hadoop/common/trunk/hadoop-common-project/hadoop-common/CHANGES.txt