Skip to content

HBASE-27212 Implement a new table based replication queue storage and… - #4672

Merged
Apache9 merged 1 commit into
apache:HBASE-27109/table_based_rqsfrom
Apache9:HBASE-27212
Aug 15, 2022
Merged

HBASE-27212 Implement a new table based replication queue storage and…#4672
Apache9 merged 1 commit into
apache:HBASE-27109/table_based_rqsfrom
Apache9:HBASE-27212

Conversation

@Apache9

Copy link
Copy Markdown
Contributor

… make the minimum replication system work

@Apache9Apache9 self-assigned this Aug 1, 2022
@Apache9
Apache9 marked this pull request as draft August 1, 2022 15:36
@Apache9

Copy link
Copy Markdown
ContributorAuthor

Finally I could pass compilation. In this PR I tried to make claimQueue basically work so at least the failover UTs without replication can pass, and I also we could implement some low level UTs for claimQueue.
Still need to add more UTs in TestReplicationSourceManager, and maybe we still need to disable more UTs to make the pre commit pass, so set it to draft for now.

@Apache-HBase

Copy link
Copy Markdown

💔 -1 overall

VoteSubsystemRuntimeComment
+0 🆗reexec0m 26sDocker mode activated.
_ Prechecks _
+1 💚dupname0m 0sNo case conflicting files found.
+0 🆗prototool0m 0sprototool was not available.
+1 💚hbaseanti0m 0sPatch does not have any anti-patterns.
+1 💚@author0m 0sThe patch does not contain any @author tags.
_ master Compile Tests _
+0 🆗mvndep0m 11sMaven dependency ordering for branch
+1 💚mvninstall2m 21smaster passed
+1 💚compile3m 27smaster passed
+1 💚checkstyle0m 48smaster passed
+1 💚spotless0m 44sbranch has no errors when running spotless:check.
+1 💚spotbugs4m 32smaster passed
_ Patch Compile Tests _
+0 🆗mvndep0m 12sMaven dependency ordering for patch
+1 💚mvninstall2m 14sthe patch passed
-1 ❌compile2m 10shbase-server in the patch failed.
-0 ⚠️cc2m 10shbase-server in the patch failed.
-0 ⚠️javac2m 10shbase-server in the patch failed.
-0 ⚠️checkstyle0m 30shbase-server: The patch generated 2 new + 41 unchanged - 0 fixed = 43 total (was 41)
+1 💚whitespace0m 0sThe patch has no whitespace issues.
+1 💚hadoopcheck11m 22sPatch does not cause any errors with Hadoop 3.1.2 3.2.2 3.3.1.
+1 💚hbaseprotoc1m 18sthe patch passed
+1 💚spotless0m 42spatch has no errors when running spotless:check.
+1 💚spotbugs4m 39sthe patch passed
_ Other Tests _
+1 💚asflicense0m 28sThe patch does not generate ASF License warnings.
43m 8s
SubsystemReport/Notes
DockerClientAPI=1.41 ServerAPI=1.41 base: https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4672/1/artifact/yetus-general-check/output/Dockerfile
GITHUB PR#4672
Optional Testsdupname asflicense cc hbaseprotoc spotless prototool javac spotbugs hadoopcheck hbaseanti checkstyle compile
unameLinux 5e855c0f68b3 5.4.0-90-generic #101-Ubuntu SMP Fri Oct 15 20:00:55 UTC 2021 x86_64 x86_64 x86_64 GNU/Linux
Build toolmaven
Personalitydev-support/hbase-personality.sh
git revisionmaster / e8c14ee
Default JavaAdoptOpenJDK-1.8.0_282-b08
compilehttps://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4672/1/artifact/yetus-general-check/output/patch-compile-hbase-server.txt
cchttps://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4672/1/artifact/yetus-general-check/output/patch-compile-hbase-server.txt
javachttps://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4672/1/artifact/yetus-general-check/output/patch-compile-hbase-server.txt
checkstylehttps://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4672/1/artifact/yetus-general-check/output/diff-checkstyle-hbase-server.txt
Max. process+thread count66 (vs. ulimit of 30000)
modulesC: hbase-protocol-shaded hbase-replication hbase-server U: .
Console outputhttps://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4672/1/console
versionsgit=2.17.1 maven=3.6.3 spotbugs=4.2.2
Powered byApache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.

@Apache-HBase

Copy link
Copy Markdown

🎊 +1 overall

VoteSubsystemRuntimeComment
+0 🆗reexec2m 37sDocker mode activated.
_ Prechecks _
+1 💚dupname0m 1sNo case conflicting files found.
+0 🆗prototool0m 1sprototool was not available.
+1 💚hbaseanti0m 0sPatch does not have any anti-patterns.
+1 💚@author0m 0sThe patch does not contain any @author tags.
_ master Compile Tests _
+0 🆗mvndep0m 12sMaven dependency ordering for branch
+1 💚mvninstall2m 15smaster passed
+1 💚compile3m 27smaster passed
+1 💚checkstyle0m 46smaster passed
+1 💚spotless0m 43sbranch has no errors when running spotless:check.
+1 💚spotbugs4m 25smaster passed
_ Patch Compile Tests _
+0 🆗mvndep0m 11sMaven dependency ordering for patch
+1 💚mvninstall2m 14sthe patch passed
+1 💚compile3m 27sthe patch passed
+1 💚cc3m 27sthe patch passed
-0 ⚠️javac0m 17shbase-replication generated 1 new + 2 unchanged - 0 fixed = 3 total (was 2)
-0 ⚠️javac2m 13shbase-server generated 1 new + 192 unchanged - 1 fixed = 193 total (was 193)
-0 ⚠️checkstyle0m 31shbase-server: The patch generated 3 new + 41 unchanged - 0 fixed = 44 total (was 41)
+1 💚whitespace0m 0sThe patch has no whitespace issues.
+1 💚hadoopcheck11m 29sPatch does not cause any errors with Hadoop 3.1.2 3.2.2 3.3.1.
+1 💚hbaseprotoc1m 18sthe patch passed
+1 💚spotless0m 43spatch has no errors when running spotless:check.
+1 💚spotbugs4m 37sthe patch passed
_ Other Tests _
+1 💚asflicense0m 27sThe patch does not generate ASF License warnings.
45m 14s
SubsystemReport/Notes
DockerClientAPI=1.41 ServerAPI=1.41 base: https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4672/2/artifact/yetus-general-check/output/Dockerfile
GITHUB PR#4672
Optional Testsdupname asflicense cc hbaseprotoc spotless prototool javac spotbugs hadoopcheck hbaseanti checkstyle compile
unameLinux fb0e19fa74ce 5.4.0-90-generic #101-Ubuntu SMP Fri Oct 15 20:00:55 UTC 2021 x86_64 x86_64 x86_64 GNU/Linux
Build toolmaven
Personalitydev-support/hbase-personality.sh
git revisionmaster / e8c14ee
Default JavaAdoptOpenJDK-1.8.0_282-b08
javachttps://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4672/2/artifact/yetus-general-check/output/diff-compile-javac-hbase-replication.txt
javachttps://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4672/2/artifact/yetus-general-check/output/diff-compile-javac-hbase-server.txt
checkstylehttps://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4672/2/artifact/yetus-general-check/output/diff-checkstyle-hbase-server.txt
Max. process+thread count64 (vs. ulimit of 30000)
modulesC: hbase-protocol-shaded hbase-replication hbase-server U: .
Console outputhttps://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4672/2/console
versionsgit=2.17.1 maven=3.6.3 spotbugs=4.2.2
Powered byApache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.

@Apache-HBase

Copy link
Copy Markdown

💔 -1 overall

VoteSubsystemRuntimeComment
+0 🆗reexec0m 33sDocker mode activated.
_ Prechecks _
+1 💚dupname0m 0sNo case conflicting files found.
+0 🆗prototool0m 0sprototool was not available.
+1 💚hbaseanti0m 0sPatch does not have any anti-patterns.
+1 💚@author0m 0sThe patch does not contain any @author tags.
_ master Compile Tests _
+0 🆗mvndep0m 18sMaven dependency ordering for branch
+1 💚mvninstall2m 18smaster passed
+1 💚compile3m 27smaster passed
+1 💚checkstyle0m 47smaster passed
+1 💚spotless0m 43sbranch has no errors when running spotless:check.
+1 💚spotbugs4m 17smaster passed
_ Patch Compile Tests _
+0 🆗mvndep0m 11sMaven dependency ordering for patch
+1 💚mvninstall2m 12sthe patch passed
+1 💚compile3m 26sthe patch passed
+1 💚cc3m 26sthe patch passed
-0 ⚠️javac0m 16shbase-replication generated 1 new + 2 unchanged - 0 fixed = 3 total (was 2)
-0 ⚠️javac2m 12shbase-server generated 1 new + 192 unchanged - 1 fixed = 193 total (was 193)
-0 ⚠️checkstyle0m 31shbase-server: The patch generated 3 new + 42 unchanged - 0 fixed = 45 total (was 42)
+1 💚whitespace0m 0sThe patch has no whitespace issues.
+1 💚hadoopcheck11m 29sPatch does not cause any errors with Hadoop 3.1.2 3.2.2 3.3.1.
+1 💚hbaseprotoc1m 17sthe patch passed
+1 💚spotless0m 42spatch has no errors when running spotless:check.
-1 ❌spotbugs1m 25shbase-server generated 1 new + 0 unchanged - 0 fixed = 1 total (was 0)
_ Other Tests _
+1 💚asflicense0m 27sThe patch does not generate ASF License warnings.
42m 55s
ReasonTests
FindBugsmodule:hbase-server
org.apache.hadoop.hbase.master.replication.ReplicationPeerManager$1.initialize() calls Thread.sleep() with a lock held At ReplicationPeerManager.java:lock held At ReplicationPeerManager.java:[line 625]
SubsystemReport/Notes
DockerClientAPI=1.41 ServerAPI=1.41 base: https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4672/3/artifact/yetus-general-check/output/Dockerfile
GITHUB PR#4672
Optional Testsdupname asflicense cc hbaseprotoc spotless prototool javac spotbugs hadoopcheck hbaseanti checkstyle compile
unameLinux fff78d9232af 5.4.0-109-generic #123-Ubuntu SMP Fri Apr 8 09:10:54 UTC 2022 x86_64 x86_64 x86_64 GNU/Linux
Build toolmaven
Personalitydev-support/hbase-personality.sh
git revisionmaster / d734acc
Default JavaAdoptOpenJDK-1.8.0_282-b08
javachttps://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4672/3/artifact/yetus-general-check/output/diff-compile-javac-hbase-replication.txt
javachttps://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4672/3/artifact/yetus-general-check/output/diff-compile-javac-hbase-server.txt
checkstylehttps://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4672/3/artifact/yetus-general-check/output/diff-checkstyle-hbase-server.txt
spotbugshttps://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4672/3/artifact/yetus-general-check/output/new-spotbugs-hbase-server.html
Max. process+thread count64 (vs. ulimit of 30000)
modulesC: hbase-protocol-shaded hbase-replication hbase-server U: .
Console outputhttps://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4672/3/console
versionsgit=2.17.1 maven=3.6.3 spotbugs=4.2.2
Powered byApache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.

@Apache-HBase

Copy link
Copy Markdown

🎊 +1 overall

VoteSubsystemRuntimeComment
+0 🆗reexec0m 28sDocker mode activated.
_ Prechecks _
+1 💚dupname0m 1sNo case conflicting files found.
+0 🆗prototool0m 1sprototool was not available.
+1 💚hbaseanti0m 0sPatch does not have any anti-patterns.
+1 💚@author0m 0sThe patch does not contain any @author tags.
_ master Compile Tests _
+0 🆗mvndep0m 10sMaven dependency ordering for branch
+1 💚mvninstall2m 20smaster passed
+1 💚compile3m 27smaster passed
+1 💚checkstyle0m 48smaster passed
+1 💚spotless0m 45sbranch has no errors when running spotless:check.
+1 💚spotbugs4m 21smaster passed
_ Patch Compile Tests _
+0 🆗mvndep0m 10sMaven dependency ordering for patch
+1 💚mvninstall2m 16sthe patch passed
+1 💚compile3m 26sthe patch passed
+1 💚cc3m 26sthe patch passed
-0 ⚠️javac0m 17shbase-replication generated 1 new + 2 unchanged - 0 fixed = 3 total (was 2)
-0 ⚠️javac2m 12shbase-server generated 2 new + 191 unchanged - 2 fixed = 193 total (was 193)
-0 ⚠️checkstyle0m 32shbase-server: The patch generated 2 new + 42 unchanged - 0 fixed = 44 total (was 42)
+1 💚whitespace0m 0sThe patch has no whitespace issues.
+1 💚hadoopcheck11m 27sPatch does not cause any errors with Hadoop 3.1.2 3.2.2 3.3.1.
+1 💚hbaseprotoc1m 17sthe patch passed
+1 💚spotless0m 42spatch has no errors when running spotless:check.
+1 💚spotbugs4m 43sthe patch passed
_ Other Tests _
+1 💚asflicense0m 28sThe patch does not generate ASF License warnings.
43m 10s
SubsystemReport/Notes
DockerClientAPI=1.41 ServerAPI=1.41 base: https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4672/4/artifact/yetus-general-check/output/Dockerfile
GITHUB PR#4672
Optional Testsdupname asflicense cc hbaseprotoc spotless prototool javac spotbugs hadoopcheck hbaseanti checkstyle compile
unameLinux 71cbe2512893 5.4.0-109-generic #123-Ubuntu SMP Fri Apr 8 09:10:54 UTC 2022 x86_64 x86_64 x86_64 GNU/Linux
Build toolmaven
Personalitydev-support/hbase-personality.sh
git revisionmaster / d734acc
Default JavaAdoptOpenJDK-1.8.0_282-b08
javachttps://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4672/4/artifact/yetus-general-check/output/diff-compile-javac-hbase-replication.txt
javachttps://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4672/4/artifact/yetus-general-check/output/diff-compile-javac-hbase-server.txt
checkstylehttps://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4672/4/artifact/yetus-general-check/output/diff-checkstyle-hbase-server.txt
Max. process+thread count64 (vs. ulimit of 30000)
modulesC: hbase-protocol-shaded hbase-replication hbase-server U: .
Console outputhttps://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4672/4/console
versionsgit=2.17.1 maven=3.6.3 spotbugs=4.2.2
Powered byApache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.

@Apache-HBase

Copy link
Copy Markdown

💔 -1 overall

VoteSubsystemRuntimeComment
+0 🆗reexec1m 15sDocker mode activated.
-0 ⚠️yetus0m 3sUnprocessed flag(s): --brief-report-file --spotbugs-strict-precheck --whitespace-eol-ignore-list --whitespace-tabs-ignore-list --quick-hadoopcheck
_ Prechecks _
_ master Compile Tests _
+0 🆗mvndep0m 17sMaven dependency ordering for branch
+1 💚mvninstall2m 17smaster passed
+1 💚compile1m 24smaster passed
+1 💚shadedjars3m 46sbranch has no errors when building our shaded downstream artifacts.
+1 💚javadoc0m 46smaster passed
_ Patch Compile Tests _
+0 🆗mvndep0m 13sMaven dependency ordering for patch
+1 💚mvninstall2m 28sthe patch passed
+1 💚compile1m 23sthe patch passed
+1 💚javac1m 23sthe patch passed
+1 💚shadedjars3m 47spatch has no errors when building our shaded downstream artifacts.
-0 ⚠️javadoc0m 24shbase-server generated 11 new + 23 unchanged - 0 fixed = 34 total (was 23)
_ Other Tests _
+1 💚unit0m 30shbase-protocol-shaded in the patch passed.
+1 💚unit0m 17shbase-replication in the patch passed.
-1 ❌unit229m 7shbase-server in the patch failed.
250m 21s
SubsystemReport/Notes
DockerClientAPI=1.41 ServerAPI=1.41 base: https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4672/4/artifact/yetus-jdk8-hadoop3-check/output/Dockerfile
GITHUB PR#4672
Optional Testsunit javac javadoc shadedjars compile
unameLinux cda6cbc668ed 5.4.0-90-generic #101-Ubuntu SMP Fri Oct 15 20:00:55 UTC 2021 x86_64 x86_64 x86_64 GNU/Linux
Build toolmaven
Personalitydev-support/hbase-personality.sh
git revisionmaster / d734acc
Default JavaAdoptOpenJDK-1.8.0_282-b08
javadochttps://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4672/4/artifact/yetus-jdk8-hadoop3-check/output/diff-javadoc-javadoc-hbase-server.txt
unithttps://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4672/4/artifact/yetus-jdk8-hadoop3-check/output/patch-unit-hbase-server.txt
Test Resultshttps://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4672/4/testReport/
Max. process+thread count2420 (vs. ulimit of 30000)
modulesC: hbase-protocol-shaded hbase-replication hbase-server U: .
Console outputhttps://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4672/4/console
versionsgit=2.17.1 maven=3.6.3
Powered byApache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.

@Apache9

Copy link
Copy Markdown
ContributorAuthor

OK good, at least we could finish the UT run. Let me check the failed UTs.

@Apache-HBase

Copy link
Copy Markdown

🎊 +1 overall

VoteSubsystemRuntimeComment
+0 🆗reexec0m 25sDocker mode activated.
_ Prechecks _
+1 💚dupname0m 0sNo case conflicting files found.
+0 🆗prototool0m 0sprototool was not available.
+1 💚hbaseanti0m 0sPatch does not have any anti-patterns.
+1 💚@author0m 0sThe patch does not contain any @author tags.
_ master Compile Tests _
+0 🆗mvndep0m 11sMaven dependency ordering for branch
+1 💚mvninstall2m 10smaster passed
+1 💚compile3m 27smaster passed
+1 💚checkstyle0m 48smaster passed
+1 💚spotless0m 43sbranch has no errors when running spotless:check.
+1 💚spotbugs4m 19smaster passed
_ Patch Compile Tests _
+0 🆗mvndep0m 12sMaven dependency ordering for patch
+1 💚mvninstall2m 12sthe patch passed
+1 💚compile3m 24sthe patch passed
+1 💚cc3m 24sthe patch passed
-0 ⚠️javac0m 16shbase-replication generated 1 new + 2 unchanged - 0 fixed = 3 total (was 2)
-0 ⚠️javac2m 12shbase-server generated 1 new + 192 unchanged - 1 fixed = 193 total (was 193)
-0 ⚠️checkstyle0m 32shbase-server: The patch generated 3 new + 42 unchanged - 0 fixed = 45 total (was 42)
+1 💚whitespace0m 0sThe patch has no whitespace issues.
+1 💚hadoopcheck11m 18sPatch does not cause any errors with Hadoop 3.1.2 3.2.2 3.3.1.
+1 💚hbaseprotoc1m 17sthe patch passed
+1 💚spotless0m 44spatch has no errors when running spotless:check.
+1 💚spotbugs4m 34sthe patch passed
_ Other Tests _
+1 💚asflicense0m 26sThe patch does not generate ASF License warnings.
42m 38s
SubsystemReport/Notes
DockerClientAPI=1.41 ServerAPI=1.41 base: https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4672/5/artifact/yetus-general-check/output/Dockerfile
GITHUB PR#4672
Optional Testsdupname asflicense cc hbaseprotoc spotless prototool javac spotbugs hadoopcheck hbaseanti checkstyle compile
unameLinux bb8b4239e716 5.4.0-109-generic #123-Ubuntu SMP Fri Apr 8 09:10:54 UTC 2022 x86_64 x86_64 x86_64 GNU/Linux
Build toolmaven
Personalitydev-support/hbase-personality.sh
git revisionmaster / d734acc
Default JavaAdoptOpenJDK-1.8.0_282-b08
javachttps://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4672/5/artifact/yetus-general-check/output/diff-compile-javac-hbase-replication.txt
javachttps://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4672/5/artifact/yetus-general-check/output/diff-compile-javac-hbase-server.txt
checkstylehttps://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4672/5/artifact/yetus-general-check/output/diff-checkstyle-hbase-server.txt
Max. process+thread count64 (vs. ulimit of 30000)
modulesC: hbase-protocol-shaded hbase-replication hbase-server U: .
Console outputhttps://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4672/5/console
versionsgit=2.17.1 maven=3.6.3 spotbugs=4.2.2
Powered byApache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.

@Apache-HBase

Copy link
Copy Markdown

💔 -1 overall

VoteSubsystemRuntimeComment
+0 🆗reexec1m 5sDocker mode activated.
-0 ⚠️yetus0m 3sUnprocessed flag(s): --brief-report-file --spotbugs-strict-precheck --whitespace-eol-ignore-list --whitespace-tabs-ignore-list --quick-hadoopcheck
_ Prechecks _
_ master Compile Tests _
+0 🆗mvndep0m 11sMaven dependency ordering for branch
+1 💚mvninstall2m 39smaster passed
+1 💚compile1m 38smaster passed
+1 💚shadedjars3m 42sbranch has no errors when building our shaded downstream artifacts.
+1 💚javadoc0m 48smaster passed
_ Patch Compile Tests _
+0 🆗mvndep0m 13sMaven dependency ordering for patch
+1 💚mvninstall2m 32sthe patch passed
+1 💚compile1m 37sthe patch passed
+1 💚javac1m 37sthe patch passed
+1 💚shadedjars3m 40spatch has no errors when building our shaded downstream artifacts.
-0 ⚠️javadoc0m 26shbase-server generated 4 new + 93 unchanged - 3 fixed = 97 total (was 96)
_ Other Tests _
+1 💚unit0m 38shbase-protocol-shaded in the patch passed.
+1 💚unit0m 18shbase-replication in the patch passed.
-1 ❌unit208m 9shbase-server in the patch failed.
230m 21s
SubsystemReport/Notes
DockerClientAPI=1.41 ServerAPI=1.41 base: https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4672/5/artifact/yetus-jdk11-hadoop3-check/output/Dockerfile
GITHUB PR#4672
Optional Testsunit javac javadoc shadedjars compile
unameLinux eb3f7d8d8c68 5.4.0-90-generic #101-Ubuntu SMP Fri Oct 15 20:00:55 UTC 2021 x86_64 x86_64 x86_64 GNU/Linux
Build toolmaven
Personalitydev-support/hbase-personality.sh
git revisionmaster / d734acc
Default JavaAdoptOpenJDK-11.0.10+9
javadochttps://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4672/5/artifact/yetus-jdk11-hadoop3-check/output/diff-javadoc-javadoc-hbase-server.txt
unithttps://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4672/5/artifact/yetus-jdk11-hadoop3-check/output/patch-unit-hbase-server.txt
Test Resultshttps://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4672/5/testReport/
Max. process+thread count2557 (vs. ulimit of 30000)
modulesC: hbase-protocol-shaded hbase-replication hbase-server U: .
Console outputhttps://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4672/5/console
versionsgit=2.17.1 maven=3.6.3
Powered byApache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.

@Apache-HBase

Copy link
Copy Markdown

🎊 +1 overall

VoteSubsystemRuntimeComment
+0 🆗reexec0m 23sDocker mode activated.
_ Prechecks _
+1 💚dupname0m 0sNo case conflicting files found.
+0 🆗prototool0m 0sprototool was not available.
+1 💚hbaseanti0m 0sPatch does not have any anti-patterns.
+1 💚@author0m 0sThe patch does not contain any @author tags.
_ master Compile Tests _
+0 🆗mvndep0m 11sMaven dependency ordering for branch
+1 💚mvninstall2m 8smaster passed
+1 💚compile3m 26smaster passed
+1 💚checkstyle0m 48smaster passed
+1 💚spotless0m 43sbranch has no errors when running spotless:check.
+1 💚spotbugs4m 14smaster passed
_ Patch Compile Tests _
+0 🆗mvndep0m 11sMaven dependency ordering for patch
+1 💚mvninstall2m 19sthe patch passed
+1 💚compile3m 28sthe patch passed
+1 💚cc3m 28sthe patch passed
-0 ⚠️javac0m 16shbase-replication generated 1 new + 2 unchanged - 0 fixed = 3 total (was 2)
-0 ⚠️javac2m 15shbase-server generated 1 new + 192 unchanged - 1 fixed = 193 total (was 193)
-0 ⚠️checkstyle0m 31shbase-server: The patch generated 3 new + 42 unchanged - 0 fixed = 45 total (was 42)
+1 💚whitespace0m 0sThe patch has no whitespace issues.
+1 💚hadoopcheck11m 23sPatch does not cause any errors with Hadoop 3.1.2 3.2.2 3.3.1.
+1 💚hbaseprotoc1m 17sthe patch passed
+1 💚spotless0m 42spatch has no errors when running spotless:check.
+1 💚spotbugs4m 44sthe patch passed
_ Other Tests _
+1 💚asflicense0m 27sThe patch does not generate ASF License warnings.
42m 46s
SubsystemReport/Notes
DockerClientAPI=1.41 ServerAPI=1.41 base: https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4672/6/artifact/yetus-general-check/output/Dockerfile
GITHUB PR#4672
Optional Testsdupname asflicense cc hbaseprotoc spotless prototool javac spotbugs hadoopcheck hbaseanti checkstyle compile
unameLinux 282ca4f19103 5.4.0-109-generic #123-Ubuntu SMP Fri Apr 8 09:10:54 UTC 2022 x86_64 x86_64 x86_64 GNU/Linux
Build toolmaven
Personalitydev-support/hbase-personality.sh
git revisionmaster / 2ac5578
Default JavaAdoptOpenJDK-1.8.0_282-b08
javachttps://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4672/6/artifact/yetus-general-check/output/diff-compile-javac-hbase-replication.txt
javachttps://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4672/6/artifact/yetus-general-check/output/diff-compile-javac-hbase-server.txt
checkstylehttps://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4672/6/artifact/yetus-general-check/output/diff-checkstyle-hbase-server.txt
Max. process+thread count60 (vs. ulimit of 30000)
modulesC: hbase-protocol-shaded hbase-replication hbase-server U: .
Console outputhttps://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4672/6/console
versionsgit=2.17.1 maven=3.6.3 spotbugs=4.2.2
Powered byApache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.

@Apache-HBase

Copy link
Copy Markdown

💔 -1 overall

VoteSubsystemRuntimeComment
+0 🆗reexec1m 59sDocker mode activated.
-0 ⚠️yetus0m 3sUnprocessed flag(s): --brief-report-file --spotbugs-strict-precheck --whitespace-eol-ignore-list --whitespace-tabs-ignore-list --quick-hadoopcheck
_ Prechecks _
_ master Compile Tests _
+0 🆗mvndep0m 10sMaven dependency ordering for branch
+1 💚mvninstall2m 43smaster passed
+1 💚compile1m 36smaster passed
+1 💚shadedjars4m 8sbranch has no errors when building our shaded downstream artifacts.
+1 💚javadoc0m 45smaster passed
_ Patch Compile Tests _
+0 🆗mvndep0m 11sMaven dependency ordering for patch
+1 💚mvninstall2m 31sthe patch passed
+1 💚compile1m 30sthe patch passed
+1 💚javac1m 30sthe patch passed
+1 💚shadedjars4m 44spatch has no errors when building our shaded downstream artifacts.
-0 ⚠️javadoc0m 25shbase-server generated 4 new + 23 unchanged - 0 fixed = 27 total (was 23)
_ Other Tests _
+1 💚unit0m 32shbase-protocol-shaded in the patch passed.
+1 💚unit0m 18shbase-replication in the patch passed.
-1 ❌unit264m 39shbase-server in the patch failed.
289m 21s
SubsystemReport/Notes
DockerClientAPI=1.41 ServerAPI=1.41 base: https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4672/6/artifact/yetus-jdk8-hadoop3-check/output/Dockerfile
GITHUB PR#4672
Optional Testsunit javac javadoc shadedjars compile
unameLinux f3458562aca6 5.4.0-90-generic #101-Ubuntu SMP Fri Oct 15 20:00:55 UTC 2021 x86_64 x86_64 x86_64 GNU/Linux
Build toolmaven
Personalitydev-support/hbase-personality.sh
git revisionmaster / 2ac5578
Default JavaAdoptOpenJDK-1.8.0_282-b08
javadochttps://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4672/6/artifact/yetus-jdk8-hadoop3-check/output/diff-javadoc-javadoc-hbase-server.txt
unithttps://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4672/6/artifact/yetus-jdk8-hadoop3-check/output/patch-unit-hbase-server.txt
Test Resultshttps://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4672/6/testReport/
Max. process+thread count2330 (vs. ulimit of 30000)
modulesC: hbase-protocol-shaded hbase-replication hbase-server U: .
Console outputhttps://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4672/6/console
versionsgit=2.17.1 maven=3.6.3
Powered byApache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.

@Apache-HBase

Copy link
Copy Markdown

🎊 +1 overall

VoteSubsystemRuntimeComment
+0 🆗reexec0m 26sDocker mode activated.
_ Prechecks _
+1 💚dupname0m 0sNo case conflicting files found.
+0 🆗prototool0m 0sprototool was not available.
+1 💚hbaseanti0m 0sPatch does not have any anti-patterns.
+1 💚@author0m 0sThe patch does not contain any @author tags.
_ master Compile Tests _
+0 🆗mvndep0m 12sMaven dependency ordering for branch
+1 💚mvninstall2m 13smaster passed
+1 💚compile3m 27smaster passed
+1 💚checkstyle0m 47smaster passed
+1 💚spotless0m 43sbranch has no errors when running spotless:check.
+1 💚spotbugs4m 22smaster passed
_ Patch Compile Tests _
+0 🆗mvndep0m 11sMaven dependency ordering for patch
+1 💚mvninstall2m 15sthe patch passed
+1 💚compile3m 26sthe patch passed
+1 💚cc3m 26sthe patch passed
-0 ⚠️javac0m 17shbase-replication generated 1 new + 2 unchanged - 0 fixed = 3 total (was 2)
-0 ⚠️javac2m 13shbase-server generated 1 new + 192 unchanged - 1 fixed = 193 total (was 193)
-0 ⚠️checkstyle0m 31shbase-server: The patch generated 2 new + 42 unchanged - 0 fixed = 44 total (was 42)
+1 💚whitespace0m 0sThe patch has no whitespace issues.
+1 💚hadoopcheck11m 19sPatch does not cause any errors with Hadoop 3.1.2 3.2.2 3.3.1.
+1 💚hbaseprotoc1m 17sthe patch passed
+1 💚spotless0m 43spatch has no errors when running spotless:check.
+1 💚spotbugs5m 20sthe patch passed
_ Other Tests _
+1 💚asflicense0m 27sThe patch does not generate ASF License warnings.
43m 32s
SubsystemReport/Notes
DockerClientAPI=1.41 ServerAPI=1.41 base: https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4672/7/artifact/yetus-general-check/output/Dockerfile
GITHUB PR#4672
Optional Testsdupname asflicense cc hbaseprotoc spotless prototool javac spotbugs hadoopcheck hbaseanti checkstyle compile
unameLinux 739314e8a754 5.4.0-109-generic #123-Ubuntu SMP Fri Apr 8 09:10:54 UTC 2022 x86_64 x86_64 x86_64 GNU/Linux
Build toolmaven
Personalitydev-support/hbase-personality.sh
git revisionmaster / 2ac5578
Default JavaAdoptOpenJDK-1.8.0_282-b08
javachttps://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4672/7/artifact/yetus-general-check/output/diff-compile-javac-hbase-replication.txt
javachttps://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4672/7/artifact/yetus-general-check/output/diff-compile-javac-hbase-server.txt
checkstylehttps://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4672/7/artifact/yetus-general-check/output/diff-checkstyle-hbase-server.txt
Max. process+thread count64 (vs. ulimit of 30000)
modulesC: hbase-protocol-shaded hbase-replication hbase-server U: .
Console outputhttps://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4672/7/console
versionsgit=2.17.1 maven=3.6.3 spotbugs=4.2.2
Powered byApache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.

@Apache9
Apache9 marked this pull request as ready for review August 5, 2022 12:51
@Apache-HBase

Copy link
Copy Markdown

🎊 +1 overall

VoteSubsystemRuntimeComment
+0 🆗reexec0m 25sDocker mode activated.
_ Prechecks _
+1 💚dupname0m 1sNo case conflicting files found.
+0 🆗prototool0m 1sprototool was not available.
+1 💚hbaseanti0m 0sPatch does not have any anti-patterns.
+1 💚@author0m 0sThe patch does not contain any @author tags.
_ master Compile Tests _
+0 🆗mvndep0m 10sMaven dependency ordering for branch
+1 💚mvninstall2m 14smaster passed
+1 💚compile3m 26smaster passed
+1 💚checkstyle0m 46smaster passed
+1 💚spotless0m 42sbranch has no errors when running spotless:check.
+1 💚spotbugs4m 18smaster passed
_ Patch Compile Tests _
+0 🆗mvndep0m 11sMaven dependency ordering for patch
+1 💚mvninstall2m 16sthe patch passed
+1 💚compile3m 27sthe patch passed
+1 💚cc3m 27sthe patch passed
-0 ⚠️javac0m 16shbase-replication generated 1 new + 2 unchanged - 0 fixed = 3 total (was 2)
-0 ⚠️javac2m 15shbase-server generated 1 new + 192 unchanged - 1 fixed = 193 total (was 193)
-0 ⚠️checkstyle0m 31shbase-server: The patch generated 2 new + 42 unchanged - 0 fixed = 44 total (was 42)
+1 💚whitespace0m 0sThe patch has no whitespace issues.
+1 💚hadoopcheck11m 13sPatch does not cause any errors with Hadoop 3.1.2 3.2.2 3.3.1.
+1 💚hbaseprotoc1m 17sthe patch passed
+1 💚spotless0m 42spatch has no errors when running spotless:check.
+1 💚spotbugs4m 37sthe patch passed
_ Other Tests _
+1 💚asflicense0m 27sThe patch does not generate ASF License warnings.
42m 16s
SubsystemReport/Notes
DockerClientAPI=1.41 ServerAPI=1.41 base: https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4672/8/artifact/yetus-general-check/output/Dockerfile
GITHUB PR#4672
Optional Testsdupname asflicense cc hbaseprotoc spotless prototool javac spotbugs hadoopcheck hbaseanti checkstyle compile
unameLinux b275e2d86bf9 5.4.0-109-generic #123-Ubuntu SMP Fri Apr 8 09:10:54 UTC 2022 x86_64 x86_64 x86_64 GNU/Linux
Build toolmaven
Personalitydev-support/hbase-personality.sh
git revisionmaster / 2ac5578
Default JavaAdoptOpenJDK-1.8.0_282-b08
javachttps://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4672/8/artifact/yetus-general-check/output/diff-compile-javac-hbase-replication.txt
javachttps://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4672/8/artifact/yetus-general-check/output/diff-compile-javac-hbase-server.txt
checkstylehttps://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4672/8/artifact/yetus-general-check/output/diff-checkstyle-hbase-server.txt
Max. process+thread count64 (vs. ulimit of 30000)
modulesC: hbase-protocol-shaded hbase-replication hbase-server U: .
Console outputhttps://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4672/8/console
versionsgit=2.17.1 maven=3.6.3 spotbugs=4.2.2
Powered byApache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.

@Apache-HBase

Copy link
Copy Markdown

💔 -1 overall

VoteSubsystemRuntimeComment
+0 🆗reexec1m 6sDocker mode activated.
-0 ⚠️yetus0m 3sUnprocessed flag(s): --brief-report-file --spotbugs-strict-precheck --whitespace-eol-ignore-list --whitespace-tabs-ignore-list --quick-hadoopcheck
_ Prechecks _
_ master Compile Tests _
+0 🆗mvndep0m 16sMaven dependency ordering for branch
+1 💚mvninstall2m 43smaster passed
+1 💚compile1m 39smaster passed
+1 💚shadedjars3m 40sbranch has no errors when building our shaded downstream artifacts.
+1 💚javadoc0m 49smaster passed
_ Patch Compile Tests _
+0 🆗mvndep0m 13sMaven dependency ordering for patch
+1 💚mvninstall2m 37sthe patch passed
+1 💚compile1m 38sthe patch passed
+1 💚javac1m 38sthe patch passed
+1 💚shadedjars3m 41spatch has no errors when building our shaded downstream artifacts.
+1 💚javadoc0m 47sthe patch passed
_ Other Tests _
+1 💚unit0m 38shbase-protocol-shaded in the patch passed.
+1 💚unit0m 19shbase-replication in the patch passed.
-1 ❌unit208m 47shbase-server in the patch failed.
230m 54s
SubsystemReport/Notes
DockerClientAPI=1.41 ServerAPI=1.41 base: https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4672/8/artifact/yetus-jdk11-hadoop3-check/output/Dockerfile
GITHUB PR#4672
Optional Testsunit javac javadoc shadedjars compile
unameLinux ad85e7bd1e7a 5.4.0-90-generic #101-Ubuntu SMP Fri Oct 15 20:00:55 UTC 2021 x86_64 x86_64 x86_64 GNU/Linux
Build toolmaven
Personalitydev-support/hbase-personality.sh
git revisionmaster / 2ac5578
Default JavaAdoptOpenJDK-11.0.10+9
unithttps://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4672/8/artifact/yetus-jdk11-hadoop3-check/output/patch-unit-hbase-server.txt
Test Resultshttps://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4672/8/testReport/
Max. process+thread count2780 (vs. ulimit of 30000)
modulesC: hbase-protocol-shaded hbase-replication hbase-server U: .
Console outputhttps://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4672/8/console
versionsgit=2.17.1 maven=3.6.3
Powered byApache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.

@Apache9

Copy link
Copy Markdown
ContributorAuthor

TestRaceBetweenSCPAndTRSP also failed in other pre commit result so should not be related. TestAsyncReplicationAdminApi should be related. It is flaky in the recent pre commit build. Let me take a look.

@Apache-HBase

Copy link
Copy Markdown

🎊 +1 overall

VoteSubsystemRuntimeComment
+0 🆗reexec0m 25sDocker mode activated.
_ Prechecks _
+1 💚dupname0m 1sNo case conflicting files found.
+0 🆗prototool0m 1sprototool was not available.
+1 💚hbaseanti0m 0sPatch does not have any anti-patterns.
+1 💚@author0m 0sThe patch does not contain any @author tags.
_ master Compile Tests _
+0 🆗mvndep0m 13sMaven dependency ordering for branch
+1 💚mvninstall2m 12smaster passed
+1 💚compile3m 25smaster passed
+1 💚checkstyle0m 46smaster passed
+1 💚spotless0m 43sbranch has no errors when running spotless:check.
+1 💚spotbugs4m 15smaster passed
_ Patch Compile Tests _
+0 🆗mvndep0m 11sMaven dependency ordering for patch
+1 💚mvninstall2m 12sthe patch passed
+1 💚compile3m 28sthe patch passed
+1 💚cc3m 28sthe patch passed
-0 ⚠️javac0m 17shbase-replication generated 1 new + 2 unchanged - 0 fixed = 3 total (was 2)
-0 ⚠️javac2m 13shbase-server generated 1 new + 192 unchanged - 1 fixed = 193 total (was 193)
-0 ⚠️checkstyle0m 31shbase-server: The patch generated 2 new + 42 unchanged - 0 fixed = 44 total (was 42)
+1 💚whitespace0m 0sThe patch has no whitespace issues.
+1 💚hadoopcheck11m 22sPatch does not cause any errors with Hadoop 3.1.2 3.2.2 3.3.1.
+1 💚hbaseprotoc1m 18sthe patch passed
+1 💚spotless0m 43spatch has no errors when running spotless:check.
+1 💚spotbugs4m 42sthe patch passed
_ Other Tests _
+1 💚asflicense0m 27sThe patch does not generate ASF License warnings.
42m 56s
SubsystemReport/Notes
DockerClientAPI=1.41 ServerAPI=1.41 base: https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4672/9/artifact/yetus-general-check/output/Dockerfile
GITHUB PR#4672
Optional Testsdupname asflicense cc hbaseprotoc spotless prototool javac spotbugs hadoopcheck hbaseanti checkstyle compile
unameLinux 786afd963c08 5.4.0-109-generic #123-Ubuntu SMP Fri Apr 8 09:10:54 UTC 2022 x86_64 x86_64 x86_64 GNU/Linux
Build toolmaven
Personalitydev-support/hbase-personality.sh
git revisionmaster / f8dcf07
Default JavaAdoptOpenJDK-1.8.0_282-b08
javachttps://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4672/9/artifact/yetus-general-check/output/diff-compile-javac-hbase-replication.txt
javachttps://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4672/9/artifact/yetus-general-check/output/diff-compile-javac-hbase-server.txt
checkstylehttps://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4672/9/artifact/yetus-general-check/output/diff-checkstyle-hbase-server.txt
Max. process+thread count64 (vs. ulimit of 30000)
modulesC: hbase-protocol-shaded hbase-replication hbase-server U: .
Console outputhttps://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4672/9/console
versionsgit=2.17.1 maven=3.6.3 spotbugs=4.2.2
Powered byApache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.

@Apache-HBase

Copy link
Copy Markdown

🎊 +1 overall

VoteSubsystemRuntimeComment
+0 🆗reexec1m 16sDocker mode activated.
-0 ⚠️yetus0m 3sUnprocessed flag(s): --brief-report-file --spotbugs-strict-precheck --whitespace-eol-ignore-list --whitespace-tabs-ignore-list --quick-hadoopcheck
_ Prechecks _
_ master Compile Tests _
+0 🆗mvndep0m 14sMaven dependency ordering for branch
+1 💚mvninstall3m 5smaster passed
+1 💚compile1m 50smaster passed
+1 💚shadedjars4m 11sbranch has no errors when building our shaded downstream artifacts.
+1 💚javadoc0m 50smaster passed
_ Patch Compile Tests _
+0 🆗mvndep0m 11sMaven dependency ordering for patch
+1 💚mvninstall3m 1sthe patch passed
+1 💚compile1m 54sthe patch passed
+1 💚javac1m 54sthe patch passed
+1 💚shadedjars4m 24spatch has no errors when building our shaded downstream artifacts.
+1 💚javadoc0m 53sthe patch passed
_ Other Tests _
+1 💚unit0m 42shbase-protocol-shaded in the patch passed.
+1 💚unit0m 17shbase-replication in the patch passed.
+1 💚unit240m 10shbase-server in the patch passed.
264m 43s
SubsystemReport/Notes
DockerClientAPI=1.41 ServerAPI=1.41 base: https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4672/9/artifact/yetus-jdk11-hadoop3-check/output/Dockerfile
GITHUB PR#4672
Optional Testsunit javac javadoc shadedjars compile
unameLinux 014793d3fd11 5.4.0-90-generic #101-Ubuntu SMP Fri Oct 15 20:00:55 UTC 2021 x86_64 x86_64 x86_64 GNU/Linux
Build toolmaven
Personalitydev-support/hbase-personality.sh
git revisionmaster / f8dcf07
Default JavaAdoptOpenJDK-11.0.10+9
Test Resultshttps://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4672/9/testReport/
Max. process+thread count2793 (vs. ulimit of 30000)
modulesC: hbase-protocol-shaded hbase-replication hbase-server U: .
Console outputhttps://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4672/9/console
versionsgit=2.17.1 maven=3.6.3
Powered byApache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.

@Apache-HBase

Copy link
Copy Markdown

💔 -1 overall

VoteSubsystemRuntimeComment
+0 🆗reexec1m 44sDocker mode activated.
-0 ⚠️yetus0m 3sUnprocessed flag(s): --brief-report-file --spotbugs-strict-precheck --whitespace-eol-ignore-list --whitespace-tabs-ignore-list --quick-hadoopcheck
_ Prechecks _
_ master Compile Tests _
+0 🆗mvndep0m 12sMaven dependency ordering for branch
+1 💚mvninstall2m 30smaster passed
+1 💚compile1m 23smaster passed
+1 💚shadedjars4m 15sbranch has no errors when building our shaded downstream artifacts.
+1 💚javadoc0m 45smaster passed
_ Patch Compile Tests _
+0 🆗mvndep0m 11sMaven dependency ordering for patch
+1 💚mvninstall2m 46sthe patch passed
+1 💚compile1m 42sthe patch passed
+1 💚javac1m 42sthe patch passed
+1 💚shadedjars4m 36spatch has no errors when building our shaded downstream artifacts.
+1 💚javadoc0m 43sthe patch passed
_ Other Tests _
+1 💚unit0m 33shbase-protocol-shaded in the patch passed.
+1 💚unit0m 21shbase-replication in the patch passed.
-1 ❌unit249m 27shbase-server in the patch failed.
273m 22s
SubsystemReport/Notes
DockerClientAPI=1.41 ServerAPI=1.41 base: https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4672/9/artifact/yetus-jdk8-hadoop3-check/output/Dockerfile
GITHUB PR#4672
Optional Testsunit javac javadoc shadedjars compile
unameLinux 1ba3299b888f 5.4.0-90-generic #101-Ubuntu SMP Fri Oct 15 20:00:55 UTC 2021 x86_64 x86_64 x86_64 GNU/Linux
Build toolmaven
Personalitydev-support/hbase-personality.sh
git revisionmaster / f8dcf07
Default JavaAdoptOpenJDK-1.8.0_282-b08
unithttps://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4672/9/artifact/yetus-jdk8-hadoop3-check/output/patch-unit-hbase-server.txt
Test Resultshttps://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4672/9/testReport/
Max. process+thread count2380 (vs. ulimit of 30000)
modulesC: hbase-protocol-shaded hbase-replication hbase-server U: .
Console outputhttps://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4672/9/console
versionsgit=2.17.1 maven=3.6.3
Powered byApache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.

@Apache9

Copy link
Copy Markdown
ContributorAuthor

Tried TestAsyncReplicationAdminApi several times locally, no failure... And it is not failing in the newest pre commit result.

@ddupgddupg left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM.
This is a great beginning.

// removed from peer config
for (String encodedRegionName : ReplicationBarrierFamilyFormat
.getTableEncodedRegionNamesForSerialReplication(conn, tn)) {
LOG.info("=============" + tn + ", " + encodedRegionName);

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is this log for debugging?

Copy link
Copy Markdown
ContributorAuthor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ah yes, let me remove this...

Apache9 added a commit that referenced this pull request Nov 5, 2022
… make the minimum replication system work (#4672)
Signed-off-by: Xin Sun <ddupgs@gmail.com>
Apache9 added a commit that referenced this pull request Nov 6, 2022
… make the minimum replication system work (#4672)
Signed-off-by: Xin Sun <ddupgs@gmail.com>
Apache9 added a commit that referenced this pull request Nov 13, 2022
… make the minimum replication system work (#4672)
Signed-off-by: Xin Sun <ddupgs@gmail.com>
Apache9 added a commit that referenced this pull request Nov 27, 2022
… make the minimum replication system work (#4672)
Signed-off-by: Xin Sun <ddupgs@gmail.com>
Apache9 added a commit that referenced this pull request Nov 30, 2022
… make the minimum replication system work (#4672)
Signed-off-by: Xin Sun <ddupgs@gmail.com>
Apache9 added a commit that referenced this pull request Dec 3, 2022
… make the minimum replication system work (#4672)
Signed-off-by: Xin Sun <ddupgs@gmail.com>
Apache9 added a commit that referenced this pull request Jan 10, 2023
… make the minimum replication system work (#4672)
Signed-off-by: Xin Sun <ddupgs@gmail.com>
Apache9 added a commit that referenced this pull request Jan 18, 2023
… make the minimum replication system work (#4672)
Signed-off-by: Xin Sun <ddupgs@gmail.com>
Apache9 added a commit that referenced this pull request Jan 19, 2023
… make the minimum replication system work (#4672)
Signed-off-by: Xin Sun <ddupgs@gmail.com>
Apache9 added a commit that referenced this pull request Jan 31, 2023
… make the minimum replication system work (#4672)
Signed-off-by: Xin Sun <ddupgs@gmail.com>
Apache9 added a commit that referenced this pull request Feb 3, 2023
… make the minimum replication system work (#4672)
Signed-off-by: Xin Sun <ddupgs@gmail.com>
Apache9 added a commit that referenced this pull request Mar 6, 2023
… make the minimum replication system work (#4672)
Signed-off-by: Xin Sun <ddupgs@gmail.com>
Apache9 added a commit that referenced this pull request Mar 9, 2023
… make the minimum replication system work (#4672)
Signed-off-by: Xin Sun <ddupgs@gmail.com>
Apache9 added a commit that referenced this pull request Mar 13, 2023
… make the minimum replication system work (#4672)
Signed-off-by: Xin Sun <ddupgs@gmail.com>
Apache9 added a commit that referenced this pull request Mar 15, 2023
… make the minimum replication system work (#4672)
Signed-off-by: Xin Sun <ddupgs@gmail.com>
Apache9 added a commit that referenced this pull request Mar 18, 2023
… make the minimum replication system work (#4672)
Signed-off-by: Xin Sun <ddupgs@gmail.com>
Apache9 added a commit that referenced this pull request Apr 5, 2023
… make the minimum replication system work (#4672)
Signed-off-by: Xin Sun <ddupgs@gmail.com>
Apache9 added a commit that referenced this pull request Apr 18, 2023
… make the minimum replication system work (#4672)
Signed-off-by: Xin Sun <ddupgs@gmail.com>
Apache9 added a commit that referenced this pull request Apr 21, 2023
… make the minimum replication system work (#4672)
Signed-off-by: Xin Sun <ddupgs@gmail.com>
Apache9 added a commit that referenced this pull request Apr 23, 2023
… make the minimum replication system work (#4672)
Signed-off-by: Xin Sun <ddupgs@gmail.com>
Apache9 added a commit that referenced this pull request Apr 24, 2023
… make the minimum replication system work (#4672)
Signed-off-by: Xin Sun <ddupgs@gmail.com>
Apache9 added a commit that referenced this pull request Apr 25, 2023
… make the minimum replication system work (#4672)
Signed-off-by: Xin Sun <ddupgs@gmail.com>
Apache9 added a commit that referenced this pull request Apr 27, 2023
… make the minimum replication system work (#4672)
Signed-off-by: Xin Sun <ddupgs@gmail.com>
Apache9 added a commit that referenced this pull request Apr 28, 2023
… make the minimum replication system work (#4672)
Signed-off-by: Xin Sun <ddupgs@gmail.com>
Apache9 added a commit that referenced this pull request May 1, 2023
… make the minimum replication system work (#4672)
Signed-off-by: Xin Sun <ddupgs@gmail.com>
Apache9 added a commit that referenced this pull request May 4, 2023
… make the minimum replication system work (#4672)
Signed-off-by: Xin Sun <ddupgs@gmail.com>
Apache9 added a commit that referenced this pull request May 6, 2023
… make the minimum replication system work (#4672)
Signed-off-by: Xin Sun <ddupgs@gmail.com>
Apache9 added a commit that referenced this pull request May 8, 2023
… make the minimum replication system work (#4672)
Signed-off-by: Xin Sun <ddupgs@gmail.com>
Apache9 added a commit that referenced this pull request May 13, 2023
… make the minimum replication system work (#4672)
Signed-off-by: Xin Sun <ddupgs@gmail.com>
Apache9 added a commit that referenced this pull request May 15, 2023
… make the minimum replication system work (#4672)
Signed-off-by: Xin Sun <ddupgs@gmail.com>
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

@Apache9@Apache-HBase@ddupg