-1 tests included. The patch doesn't appear to include any new or modified tests.
Please justify why no new tests are needed for this patch.
Also please list what manual steps were performed to verify this patch.
I started an RS and did a remote FS bulk load call to verify some of the logging changes.
-1 javac. The applied patch generated 5 javac compiler warnings (more than the trunk's current 4 warnings).
I don't see how this came to be. I've merely added a few already existing objects here and there and changed a few string messages. I am not sure this warning is cause of me.
-1 findbugs. The patch appears to introduce 7 new Findbugs (version 1.3.9) warnings.
No, it does not. At least I don't see how the warnings apply to my changes.
-1 core tests. The patch failed these unit tests: org.apache.hadoop.hbase.TestZooKeeper
This doesn't seem to be cause of my trivial patch either.