Skip to content

HBASE-27623 Start a new ReplicationSyncUp after the previous failed - #5150

Merged
Apache9 merged 1 commit into
apache:HBASE-27109/table_based_rqsfrom
2005hithlj:HBASE-27623
Apr 5, 2023
Merged

HBASE-27623 Start a new ReplicationSyncUp after the previous failed#5150
Apache9 merged 1 commit into
apache:HBASE-27109/table_based_rqsfrom
2005hithlj:HBASE-27623

Conversation

@2005hithlj

@2005hithlj2005hithlj commented Mar 28, 2023

Copy link
Copy Markdown
Contributor

@Apache-HBase

Apache-HBase commented Mar 28, 2023

Copy link
Copy Markdown

🎊 +1 overall

VoteSubsystemRuntimeComment
+0 🆗reexec0m 27sDocker mode activated.
_ Prechecks _
+1 💚dupname0m 0sNo case conflicting files found.
+1 💚hbaseanti0m 0sPatch does not have any anti-patterns.
+1 💚@author0m 0sThe patch does not contain any @author tags.
_ HBASE-27109/table_based_rqs Compile Tests _
+1 💚mvninstall3m 42sHBASE-27109/table_based_rqs passed
+1 💚compile2m 20sHBASE-27109/table_based_rqs passed
+1 💚checkstyle0m 31sHBASE-27109/table_based_rqs passed
+1 💚spotless0m 40sbranch has no errors when running spotless:check.
+1 💚spotbugs1m 23sHBASE-27109/table_based_rqs passed
_ Patch Compile Tests _
+1 💚mvninstall3m 19sthe patch passed
+1 💚compile2m 19sthe patch passed
+1 💚javac2m 19sthe patch passed
-0 ⚠️checkstyle0m 33shbase-server: The patch generated 1 new + 0 unchanged - 0 fixed = 1 total (was 0)
+1 💚whitespace0m 0sThe patch has no whitespace issues.
+1 💚hadoopcheck12m 52sPatch does not cause any errors with Hadoop 3.2.4 3.3.4.
+1 💚spotless0m 37spatch has no errors when running spotless:check.
+1 💚spotbugs1m 28sthe patch passed
_ Other Tests _
+1 💚asflicense0m 10sThe patch does not generate ASF License warnings.
38m 9s
SubsystemReport/Notes
DockerClientAPI=1.42 ServerAPI=1.42 base: https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5150/1/artifact/yetus-general-check/output/Dockerfile
GITHUB PR#5150
Optional Testsdupname asflicense javac spotbugs hadoopcheck hbaseanti spotless checkstyle compile
unameLinux d0a1765e25a0 5.4.0-1094-aws #102~18.04.1-Ubuntu SMP Tue Jan 10 21:07:03 UTC 2023 x86_64 x86_64 x86_64 GNU/Linux
Build toolmaven
Personalitydev-support/hbase-personality.sh
git revisionHBASE-27109/table_based_rqs / 49589ef
Default JavaEclipse Adoptium-11.0.17+8
checkstylehttps://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5150/1/artifact/yetus-general-check/output/diff-checkstyle-hbase-server.txt
Max. process+thread count84 (vs. ulimit of 30000)
modulesC: hbase-server U: hbase-server
Console outputhttps://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5150/1/console
versionsgit=2.34.1 maven=3.8.6 spotbugs=4.7.3
Powered byApache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.

@Apache-HBase

Apache-HBase commented Mar 28, 2023

Copy link
Copy Markdown

💔 -1 overall

VoteSubsystemRuntimeComment
+0 🆗reexec1m 0sDocker mode activated.
-0 ⚠️yetus0m 3sUnprocessed flag(s): --brief-report-file --spotbugs-strict-precheck --whitespace-eol-ignore-list --whitespace-tabs-ignore-list --quick-hadoopcheck
_ Prechecks _
_ HBASE-27109/table_based_rqs Compile Tests _
+1 💚mvninstall3m 2sHBASE-27109/table_based_rqs passed
+1 💚compile0m 41sHBASE-27109/table_based_rqs passed
+1 💚shadedjars4m 14sbranch has no errors when building our shaded downstream artifacts.
+1 💚javadoc0m 25sHBASE-27109/table_based_rqs passed
_ Patch Compile Tests _
+1 💚mvninstall2m 46sthe patch passed
+1 💚compile0m 40sthe patch passed
+1 💚javac0m 40sthe patch passed
+1 💚shadedjars4m 18spatch has no errors when building our shaded downstream artifacts.
+1 💚javadoc0m 23sthe patch passed
_ Other Tests _
-1 ❌unit257m 24shbase-server in the patch failed.
279m 21s
SubsystemReport/Notes
DockerClientAPI=1.42 ServerAPI=1.42 base: https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5150/1/artifact/yetus-jdk8-hadoop3-check/output/Dockerfile
GITHUB PR#5150
Optional Testsjavac javadoc unit shadedjars compile
unameLinux 1e2122d355e5 5.4.0-144-generic #161-Ubuntu SMP Fri Feb 3 14:49:04 UTC 2023 x86_64 x86_64 x86_64 GNU/Linux
Build toolmaven
Personalitydev-support/hbase-personality.sh
git revisionHBASE-27109/table_based_rqs / 49589ef
Default JavaTemurin-1.8.0_352-b08
unithttps://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5150/1/artifact/yetus-jdk8-hadoop3-check/output/patch-unit-hbase-server.txt
Test Resultshttps://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5150/1/testReport/
Max. process+thread count2801 (vs. ulimit of 30000)
modulesC: hbase-server U: hbase-server
Console outputhttps://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5150/1/console
versionsgit=2.34.1 maven=3.8.6
Powered byApache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.

@Apache9

Apache9 commented Mar 29, 2023

Copy link
Copy Markdown
Contributor

Let's change the title of the jira issue? And is it possible to add a UT for this?

@2005hithlj2005hithlj changed the title HBASE-27623 Avoid starting two ReplicationSyncUp at the same timeHBASE-27623 Start a new ReplicationSyncUp after the previous failedApr 3, 2023
@Apache-HBase

Apache-HBase commented Apr 3, 2023

Copy link
Copy Markdown

🎊 +1 overall

VoteSubsystemRuntimeComment
+0 🆗reexec0m 45sDocker mode activated.
_ Prechecks _
+1 💚dupname0m 0sNo case conflicting files found.
+1 💚hbaseanti0m 0sPatch does not have any anti-patterns.
+1 💚@author0m 0sThe patch does not contain any @author tags.
_ HBASE-27109/table_based_rqs Compile Tests _
+1 💚mvninstall4m 47sHBASE-27109/table_based_rqs passed
+1 💚compile2m 44sHBASE-27109/table_based_rqs passed
+1 💚checkstyle0m 38sHBASE-27109/table_based_rqs passed
+1 💚spotless0m 45sbranch has no errors when running spotless:check.
+1 💚spotbugs1m 36sHBASE-27109/table_based_rqs passed
_ Patch Compile Tests _
+1 💚mvninstall3m 51sthe patch passed
+1 💚compile2m 36sthe patch passed
+1 💚javac2m 36sthe patch passed
-0 ⚠️checkstyle0m 38shbase-server: The patch generated 1 new + 0 unchanged - 0 fixed = 1 total (was 0)
+1 💚whitespace0m 0sThe patch has no whitespace issues.
+1 💚hadoopcheck15m 42sPatch does not cause any errors with Hadoop 3.2.4 3.3.4.
+1 💚spotless0m 41spatch has no errors when running spotless:check.
+1 💚spotbugs1m 52sthe patch passed
_ Other Tests _
+1 💚asflicense0m 12sThe patch does not generate ASF License warnings.
45m 33s
SubsystemReport/Notes
DockerClientAPI=1.42 ServerAPI=1.42 base: https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5150/2/artifact/yetus-general-check/output/Dockerfile
GITHUB PR#5150
Optional Testsdupname asflicense javac spotbugs hadoopcheck hbaseanti spotless checkstyle compile
unameLinux 61d94a459000 5.4.0-1094-aws #102~18.04.1-Ubuntu SMP Tue Jan 10 21:07:03 UTC 2023 x86_64 x86_64 x86_64 GNU/Linux
Build toolmaven
Personalitydev-support/hbase-personality.sh
git revisionHBASE-27109/table_based_rqs / 49589ef
Default JavaEclipse Adoptium-11.0.17+8
checkstylehttps://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5150/2/artifact/yetus-general-check/output/diff-checkstyle-hbase-server.txt
Max. process+thread count84 (vs. ulimit of 30000)
modulesC: hbase-server U: hbase-server
Console outputhttps://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5150/2/console
versionsgit=2.34.1 maven=3.8.6 spotbugs=4.7.3
Powered byApache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.

@Apache-HBase

Apache-HBase commented Apr 3, 2023

Copy link
Copy Markdown

💔 -1 overall

VoteSubsystemRuntimeComment
+0 🆗reexec0m 31sDocker mode activated.
-0 ⚠️yetus0m 3sUnprocessed flag(s): --brief-report-file --spotbugs-strict-precheck --whitespace-eol-ignore-list --whitespace-tabs-ignore-list --quick-hadoopcheck
_ Prechecks _
_ HBASE-27109/table_based_rqs Compile Tests _
+1 💚mvninstall4m 0sHBASE-27109/table_based_rqs passed
+1 💚compile0m 46sHBASE-27109/table_based_rqs passed
+1 💚shadedjars4m 56sbranch has no errors when building our shaded downstream artifacts.
+1 💚javadoc0m 24sHBASE-27109/table_based_rqs passed
_ Patch Compile Tests _
+1 💚mvninstall3m 29sthe patch passed
+1 💚compile0m 45sthe patch passed
+1 💚javac0m 45sthe patch passed
+1 💚shadedjars4m 48spatch has no errors when building our shaded downstream artifacts.
+1 💚javadoc0m 22sthe patch passed
_ Other Tests _
-1 ❌unit215m 17shbase-server in the patch failed.
239m 40s
SubsystemReport/Notes
DockerClientAPI=1.42 ServerAPI=1.42 base: https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5150/2/artifact/yetus-jdk11-hadoop3-check/output/Dockerfile
GITHUB PR#5150
Optional Testsjavac javadoc unit shadedjars compile
unameLinux 7f7937e42551 5.4.0-1097-aws #105~18.04.1-Ubuntu SMP Mon Feb 13 17:50:57 UTC 2023 x86_64 x86_64 x86_64 GNU/Linux
Build toolmaven
Personalitydev-support/hbase-personality.sh
git revisionHBASE-27109/table_based_rqs / 49589ef
Default JavaEclipse Adoptium-11.0.17+8
unithttps://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5150/2/artifact/yetus-jdk11-hadoop3-check/output/patch-unit-hbase-server.txt
Test Resultshttps://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5150/2/testReport/
Max. process+thread count5480 (vs. ulimit of 30000)
modulesC: hbase-server U: hbase-server
Console outputhttps://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5150/2/console
versionsgit=2.34.1 maven=3.8.6
Powered byApache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.

@Apache-HBase

Apache-HBase commented Apr 4, 2023

Copy link
Copy Markdown

🎊 +1 overall

VoteSubsystemRuntimeComment
+0 🆗reexec0m 27sDocker mode activated.
_ Prechecks _
+1 💚dupname0m 0sNo case conflicting files found.
+1 💚hbaseanti0m 0sPatch does not have any anti-patterns.
+1 💚@author0m 0sThe patch does not contain any @author tags.
_ HBASE-27109/table_based_rqs Compile Tests _
+1 💚mvninstall3m 14sHBASE-27109/table_based_rqs passed
+1 💚compile2m 19sHBASE-27109/table_based_rqs passed
+1 💚checkstyle0m 34sHBASE-27109/table_based_rqs passed
+1 💚spotless0m 37sbranch has no errors when running spotless:check.
+1 💚spotbugs1m 21sHBASE-27109/table_based_rqs passed
_ Patch Compile Tests _
+1 💚mvninstall3m 15sthe patch passed
+1 💚compile2m 21sthe patch passed
+1 💚javac2m 21sthe patch passed
-0 ⚠️checkstyle0m 31shbase-server: The patch generated 1 new + 0 unchanged - 0 fixed = 1 total (was 0)
+1 💚whitespace0m 0sThe patch has no whitespace issues.
+1 💚hadoopcheck12m 50sPatch does not cause any errors with Hadoop 3.2.4 3.3.4.
+1 💚spotless0m 38spatch has no errors when running spotless:check.
+1 💚spotbugs1m 29sthe patch passed
_ Other Tests _
+1 💚asflicense0m 8sThe patch does not generate ASF License warnings.
37m 18s
SubsystemReport/Notes
DockerClientAPI=1.42 ServerAPI=1.42 base: https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5150/3/artifact/yetus-general-check/output/Dockerfile
GITHUB PR#5150
Optional Testsdupname asflicense javac spotbugs hadoopcheck hbaseanti spotless checkstyle compile
unameLinux 4a440d8fee20 5.4.0-1094-aws #102~18.04.1-Ubuntu SMP Tue Jan 10 21:07:03 UTC 2023 x86_64 x86_64 x86_64 GNU/Linux
Build toolmaven
Personalitydev-support/hbase-personality.sh
git revisionHBASE-27109/table_based_rqs / 49589ef
Default JavaEclipse Adoptium-11.0.17+8
checkstylehttps://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5150/3/artifact/yetus-general-check/output/diff-checkstyle-hbase-server.txt
Max. process+thread count79 (vs. ulimit of 30000)
modulesC: hbase-server U: hbase-server
Console outputhttps://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5150/3/console
versionsgit=2.34.1 maven=3.8.6 spotbugs=4.7.3
Powered byApache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.

@2005hithlj

2005hithlj commented Apr 4, 2023

Copy link
Copy Markdown
ContributorAuthor

@Apache9 sir.
I have changed the title and added UT. Recently, I found that the pr-merge operation is very slow to execute.

@Apache-HBase

Apache-HBase commented Apr 4, 2023

Copy link
Copy Markdown

💔 -1 overall

VoteSubsystemRuntimeComment
+0 🆗reexec5m 49sDocker mode activated.
-0 ⚠️yetus0m 4sUnprocessed flag(s): --brief-report-file --spotbugs-strict-precheck --whitespace-eol-ignore-list --whitespace-tabs-ignore-list --quick-hadoopcheck
_ Prechecks _
_ HBASE-27109/table_based_rqs Compile Tests _
+1 💚mvninstall2m 56sHBASE-27109/table_based_rqs passed
+1 💚compile0m 39sHBASE-27109/table_based_rqs passed
+1 💚shadedjars4m 14sbranch has no errors when building our shaded downstream artifacts.
+1 💚javadoc0m 24sHBASE-27109/table_based_rqs passed
_ Patch Compile Tests _
+1 💚mvninstall2m 44sthe patch passed
+1 💚compile0m 39sthe patch passed
+1 💚javac0m 39sthe patch passed
+1 💚shadedjars4m 16spatch has no errors when building our shaded downstream artifacts.
+1 💚javadoc0m 23sthe patch passed
_ Other Tests _
-1 ❌unit228m 48shbase-server in the patch failed.
254m 28s
SubsystemReport/Notes
DockerClientAPI=1.42 ServerAPI=1.42 base: https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5150/3/artifact/yetus-jdk8-hadoop3-check/output/Dockerfile
GITHUB PR#5150
Optional Testsjavac javadoc unit shadedjars compile
unameLinux 7a3cf3e27db7 5.4.0-144-generic #161-Ubuntu SMP Fri Feb 3 14:49:04 UTC 2023 x86_64 x86_64 x86_64 GNU/Linux
Build toolmaven
Personalitydev-support/hbase-personality.sh
git revisionHBASE-27109/table_based_rqs / 49589ef
Default JavaTemurin-1.8.0_352-b08
unithttps://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5150/3/artifact/yetus-jdk8-hadoop3-check/output/patch-unit-hbase-server.txt
Test Resultshttps://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5150/3/testReport/
Max. process+thread count2776 (vs. ulimit of 30000)
modulesC: hbase-server U: hbase-server
Console outputhttps://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5150/3/console
versionsgit=2.34.1 maven=3.8.6
Powered byApache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.

@Apache9

Apache9 commented Apr 5, 2023

Copy link
Copy Markdown
Contributor

The failed UT is TestClusterScopeQuotaThrottle, which should not be related, at least not related to the changes here.

Let me merge.

@Apache9
Apache9 merged commit ba5488b into apache:HBASE-27109/table_based_rqsApr 5, 2023
Apache9 pushed a commit that referenced this pull request Apr 5, 2023
@2005hithlj@Apache9
HBASE-27623Start a new ReplicationSyncUp after the previous failed (#…
f78fe59
…5150)
Signed-off-by: Duo Zhang <zhangduo@apache.org>
Apache9 pushed a commit that referenced this pull request Apr 18, 2023
@2005hithlj@Apache9
HBASE-27623Start a new ReplicationSyncUp after the previous failed (#…
845ad90
…5150)
Signed-off-by: Duo Zhang <zhangduo@apache.org>
Apache9 pushed a commit that referenced this pull request Apr 21, 2023
@2005hithlj@Apache9
HBASE-27623Start a new ReplicationSyncUp after the previous failed (#…
e4f34a2
…5150)
Signed-off-by: Duo Zhang <zhangduo@apache.org>
Apache9 pushed a commit that referenced this pull request Apr 23, 2023
@2005hithlj@Apache9
HBASE-27623Start a new ReplicationSyncUp after the previous failed (#…
7c87504
…5150)
Signed-off-by: Duo Zhang <zhangduo@apache.org>
Apache9 pushed a commit that referenced this pull request Apr 24, 2023
@2005hithlj@Apache9
HBASE-27623Start a new ReplicationSyncUp after the previous failed (#…
bfe5e19
…5150)
Signed-off-by: Duo Zhang <zhangduo@apache.org>
Apache9 pushed a commit that referenced this pull request Apr 25, 2023
@2005hithlj@Apache9
HBASE-27623Start a new ReplicationSyncUp after the previous failed (#…
941aed1
…5150)
Signed-off-by: Duo Zhang <zhangduo@apache.org>
Apache9 pushed a commit that referenced this pull request Apr 27, 2023
@2005hithlj@Apache9
HBASE-27623Start a new ReplicationSyncUp after the previous failed (#…
3c74d11
…5150)
Signed-off-by: Duo Zhang <zhangduo@apache.org>
Apache9 pushed a commit that referenced this pull request Apr 28, 2023
@2005hithlj@Apache9
HBASE-27623Start a new ReplicationSyncUp after the previous failed (#…
5a3d934
…5150)
Signed-off-by: Duo Zhang <zhangduo@apache.org>
Apache9 pushed a commit that referenced this pull request May 1, 2023
@2005hithlj@Apache9
HBASE-27623Start a new ReplicationSyncUp after the previous failed (#…
4de5361
…5150)
Signed-off-by: Duo Zhang <zhangduo@apache.org>
Apache9 pushed a commit that referenced this pull request May 4, 2023
@2005hithlj@Apache9
HBASE-27623Start a new ReplicationSyncUp after the previous failed (#…
0fcc0f2
…5150)
Signed-off-by: Duo Zhang <zhangduo@apache.org>
Apache9 pushed a commit that referenced this pull request May 6, 2023
@2005hithlj@Apache9
HBASE-27623Start a new ReplicationSyncUp after the previous failed (#…
1f38f8a
…5150)
Signed-off-by: Duo Zhang <zhangduo@apache.org>
Apache9 pushed a commit that referenced this pull request May 8, 2023
@2005hithlj@Apache9
HBASE-27623Start a new ReplicationSyncUp after the previous failed (#…
ef6431f
…5150)
Signed-off-by: Duo Zhang <zhangduo@apache.org>
Apache9 pushed a commit that referenced this pull request May 13, 2023
@2005hithlj@Apache9
HBASE-27623Start a new ReplicationSyncUp after the previous failed (#…
b8c01fc
…5150)
Signed-off-by: Duo Zhang <zhangduo@apache.org>
Apache9 pushed a commit that referenced this pull request May 14, 2023
@2005hithlj@Apache9
HBASE-27623Start a new ReplicationSyncUp after the previous failed (#…
252c4d7
…5150)
Signed-off-by: Duo Zhang <zhangduo@apache.org>
Apache9 pushed a commit that referenced this pull request May 15, 2023
@2005hithlj@Apache9
HBASE-27623Start a new ReplicationSyncUp after the previous failed (#…
650350a
…5150)
Signed-off-by: Duo Zhang <zhangduo@apache.org>
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@Apache9Apache9Apache9 approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@2005hithlj@Apache-HBase@Apache9