Hudson added a comment - 28/Sep/18 11:13 Results for branch branch-1.4 build #482 on builds.a.o : -1 overall details (if available): -1 general checks – For more information see general report -1 jdk7 checks – For more information see jdk7 report -1 jdk8 hadoop2 checks – For more information see jdk8 (hadoop2) report +1 source release artifact – See build output for details.

Hudson added a comment - 28/Sep/18 10:24 Results for branch branch-1.3 build #483 on builds.a.o : -1 overall details (if available): +1 general checks – For more information see general report -1 jdk7 checks – For more information see jdk7 report -1 jdk8 hadoop2 checks – For more information see jdk8 (hadoop2) report +1 source release artifact – See build output for details.

Hudson added a comment - 28/Sep/18 01:10 SUCCESS: Integrated in Jenkins build HBase-1.3-IT #484 (See https://builds.apache.org/job/HBase-1.3-IT/484/ ) HBASE-20766 Typo in VerifyReplication error. (apurtell: rev 10e486882a79e722d44cb42259fbcad1a08a6342) (edit) hbase-server/src/main/java/org/apache/hadoop/hbase/mapreduce/replication/VerifyReplication.java

Hudson added a comment - 27/Sep/18 12:51 Results for branch master build #513 on builds.a.o : -1 overall details (if available): +1 general checks – For more information see general report -1 jdk8 hadoop2 checks – For more information see jdk8 (hadoop2) report -1 jdk8 hadoop3 checks – For more information see jdk8 (hadoop3) report +1 source release artifact – See build output for details. +1 client integration test

Hudson added a comment - 27/Sep/18 10:48 Results for branch branch-1 build #478 on builds.a.o : -1 overall details (if available): -1 general checks – For more information see general report -1 jdk7 checks – For more information see jdk7 report -1 jdk8 hadoop2 checks – For more information see jdk8 (hadoop2) report -1 source release artifact – See build output for details.

Hudson added a comment - 27/Sep/18 04:17 Results for branch branch-2 build #1307 on builds.a.o : -1 overall details (if available): +1 general checks – For more information see general report -1 jdk8 hadoop2 checks – For more information see jdk8 (hadoop2) report -1 jdk8 hadoop3 checks – For more information see jdk8 (hadoop3) report +1 source release artifact – See build output for details. +1 client integration test

Sean Busbey added a comment - 26/Sep/18 21:58 pushed to each of the next minor releases. Thanks!

Sean Busbey added a comment - 26/Sep/18 21:50 | Vote | Subsystem | Runtime | Comment ============================================================================ | | | | Prechecks | 0 | findbugs | 0m 00s | Findbugs executables are not available. | +1 | hbaseanti | 0m 00s | Patch does not have any anti-patterns. | +1 | @author | 0m 00s | The patch does not contain any @author | | | | tags. | -1 | test4tests | 0m 00s | 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. | | | | master Compile Tests | +1 | mvninstall | 6m 17s | master passed | +1 | compile | 0m 46s | master passed | +1 | checkstyle | 0m 24s | master passed | +1 | shadedjars | 5m 00s | branch has no errors when building our | | | | shaded downstream artifacts. | +1 | javadoc | 0m 28s | master passed | | | | Patch Compile Tests | +1 | mvninstall | 5m 35s | the patch passed | +1 | compile | 0m 46s | the patch passed | -1 | javac | 0m 46s | hbase-mapreduce generated 158 new + 0 | | | | unchanged - 158 fixed = 158 total (was | | | | 158) | +1 | checkstyle | 0m 33s | the patch passed | +1 | whitespace | 0m 00s | The patch has no whitespace issues. | +1 | shadedjars | 5m 11s | patch has no errors when building our | | | | shaded downstream artifacts. | +1 | hadoopcheck | 29m 22s | Patch does not cause any errors with | | | | Hadoop 2.7.1 2.7.2 2.7.3 2.7.4 or 3.0.0. | +1 | javadoc | 0m 27s | the patch passed | | | | Other Tests | +1 | asflicense | 0m 14s | The patch does not generate ASF License | | | | warnings. | | | 55m 23s | test4tests we don't treat as -1 and this ought not have a test because it's an error message. the javac warnings are a false positive where the maven timestamps weren't properly filtered.

Sean Busbey added a comment - 03/Sep/18 15:12 I'm not putting this in Patch Available because I think Test Patch will get the wrong thing to check. I'll post a manual run before merging.

Sean Busbey added a comment - 02/Sep/18 15:42 Thanks for the patch awked06 . It looks like a GitHub PR came in about a month before: https://github.com/apache/hbase/pull/80 My apologies that we didn't notice the PR and set an assignee here sooner.