Here's the piece we needed:
+1 @author. The patch does not contain any @author tags.
-1 tests included. The patch doesn't appear to include any new or modified tests.
Please justify why no tests are needed for this patch.
+1 javadoc. The javadoc tool did not generate any warning messages.
+1 javac. The applied patch does not increase the total number of javac compiler warnings.
+1 findbugs. The patch does not introduce any new Findbugs (version ) warnings.
Since this patch is for build code, agree unit tests are inapplicable. Therefore test-patch passes.
The unit tests are not run by normal developer invocation of test-patch, but this patch is unlikely to affect the way task-controller runs, so we'll let that pass too.
The patch makes sense to me.
+1 for code review. Please commit to branch-0.20-security and branch-0.20-security-205.