Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Revert "HBASE-26125 Backport HBASE-25401 "Add trace support for async call in…" #3560

Closed
wants to merge 1 commit into from

Conversation

taklwu
Copy link
Contributor

@taklwu taklwu commented Aug 4, 2021

Reverts #3543

… call in rpc client" to branch-2 (#3543)"

This reverts commit ca09643.
@taklwu taklwu closed this Aug 4, 2021
@taklwu
Copy link
Contributor Author

taklwu commented Aug 4, 2021

it's a clean revert , closing it .

@taklwu taklwu deleted the revert-3543-HBASE-26125 branch August 4, 2021 22:59
@Apache-HBase
Copy link

🎊 +1 overall

Vote Subsystem Runtime Comment
+0 🆗 reexec 0m 37s Docker mode activated.
_ Prechecks _
+1 💚 dupname 0m 1s No case conflicting files found.
+0 🆗 prototool 0m 0s prototool was not available.
+1 💚 hbaseanti 0m 0s Patch does not have any anti-patterns.
+1 💚 @author 0m 0s The patch does not contain any @author tags.
_ branch-2 Compile Tests _
+0 🆗 mvndep 0m 26s Maven dependency ordering for branch
+1 💚 mvninstall 3m 40s branch-2 passed
+1 💚 compile 8m 55s branch-2 passed
+1 💚 checkstyle 2m 5s branch-2 passed
+1 💚 spotbugs 15m 0s branch-2 passed
_ Patch Compile Tests _
+0 🆗 mvndep 0m 20s Maven dependency ordering for patch
+1 💚 mvninstall 3m 19s the patch passed
+1 💚 compile 8m 47s the patch passed
+1 💚 cc 8m 47s the patch passed
-0 ⚠️ javac 8m 47s root generated 2 new + 1885 unchanged - 2 fixed = 1887 total (was 1887)
-0 ⚠️ checkstyle 2m 5s root: The patch generated 4 new + 22 unchanged - 0 fixed = 26 total (was 22)
-0 ⚠️ whitespace 0m 0s The patch has 2 line(s) that end in whitespace. Use git apply --whitespace=fix <<patch_file>>. Refer https://git-scm.com/docs/git-apply
+1 💚 xml 0m 3s The patch has no ill-formed XML file.
+1 💚 hadoopcheck 11m 59s Patch does not cause any errors with Hadoop 3.1.2 3.2.1.
+1 💚 hbaseprotoc 6m 4s the patch passed
+1 💚 spotbugs 16m 13s the patch passed
_ Other Tests _
+1 💚 asflicense 1m 9s The patch does not generate ASF License warnings.
89m 42s
Subsystem Report/Notes
Docker ClientAPI=1.41 ServerAPI=1.41 base: https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3560/1/artifact/yetus-general-check/output/Dockerfile
GITHUB PR #3560
Optional Tests dupname asflicense javac spotbugs hadoopcheck hbaseanti checkstyle compile cc hbaseprotoc prototool xml
uname Linux 2e1fea3103d0 4.15.0-136-generic #140-Ubuntu SMP Thu Jan 28 05:20:47 UTC 2021 x86_64 x86_64 x86_64 GNU/Linux
Build tool maven
Personality dev-support/hbase-personality.sh
git revision branch-2 / 620806e
Default Java AdoptOpenJDK-1.8.0_282-b08
javac https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3560/1/artifact/yetus-general-check/output/diff-compile-javac-root.txt
checkstyle https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3560/1/artifact/yetus-general-check/output/diff-checkstyle-root.txt
whitespace https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3560/1/artifact/yetus-general-check/output/whitespace-eol.txt
Max. process+thread count 141 (vs. ulimit of 12500)
modules C: hbase-protocol-shaded hbase-common hbase-client hbase-server . U: .
Console output https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3560/1/console
versions git=2.17.1 maven=3.6.3 spotbugs=4.2.2
Powered by Apache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.

@Apache-HBase
Copy link

🎊 +1 overall

Vote Subsystem Runtime Comment
+0 🆗 reexec 1m 32s Docker mode activated.
-0 ⚠️ yetus 0m 6s Unprocessed flag(s): --brief-report-file --spotbugs-strict-precheck --whitespace-eol-ignore-list --whitespace-tabs-ignore-list --quick-hadoopcheck
_ Prechecks _
_ branch-2 Compile Tests _
+0 🆗 mvndep 0m 26s Maven dependency ordering for branch
+1 💚 mvninstall 4m 56s branch-2 passed
+1 💚 compile 3m 14s branch-2 passed
+1 💚 shadedjars 8m 14s branch has no errors when building our shaded downstream artifacts.
+1 💚 javadoc 5m 0s branch-2 passed
_ Patch Compile Tests _
+0 🆗 mvndep 0m 19s Maven dependency ordering for patch
+1 💚 mvninstall 4m 43s the patch passed
+1 💚 compile 3m 27s the patch passed
+1 💚 javac 3m 27s the patch passed
+1 💚 shadedjars 8m 16s patch has no errors when building our shaded downstream artifacts.
+1 💚 javadoc 5m 4s the patch passed
_ Other Tests _
+1 💚 unit 257m 13s root in the patch passed.
305m 20s
Subsystem Report/Notes
Docker ClientAPI=1.41 ServerAPI=1.41 base: https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3560/1/artifact/yetus-jdk11-hadoop3-check/output/Dockerfile
GITHUB PR #3560
Optional Tests javac javadoc unit shadedjars compile
uname Linux f5aee8c5999d 4.15.0-143-generic #147-Ubuntu SMP Wed Apr 14 16:10:11 UTC 2021 x86_64 x86_64 x86_64 GNU/Linux
Build tool maven
Personality dev-support/hbase-personality.sh
git revision branch-2 / 620806e
Default Java AdoptOpenJDK-11.0.10+9
Test Results https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3560/1/testReport/
Max. process+thread count 4279 (vs. ulimit of 12500)
modules C: hbase-protocol-shaded hbase-common hbase-client hbase-server . U: .
Console output https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3560/1/console
versions git=2.17.1 maven=3.6.3
Powered by Apache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.

@Apache-HBase
Copy link

🎊 +1 overall

Vote Subsystem Runtime Comment
+0 🆗 reexec 0m 32s Docker mode activated.
-0 ⚠️ yetus 0m 6s Unprocessed flag(s): --brief-report-file --spotbugs-strict-precheck --whitespace-eol-ignore-list --whitespace-tabs-ignore-list --quick-hadoopcheck
_ Prechecks _
_ branch-2 Compile Tests _
+0 🆗 mvndep 0m 25s Maven dependency ordering for branch
+1 💚 mvninstall 3m 43s branch-2 passed
+1 💚 compile 2m 17s branch-2 passed
+1 💚 shadedjars 6m 21s branch has no errors when building our shaded downstream artifacts.
+1 💚 javadoc 3m 47s branch-2 passed
_ Patch Compile Tests _
+0 🆗 mvndep 0m 21s Maven dependency ordering for patch
+1 💚 mvninstall 3m 23s the patch passed
+1 💚 compile 2m 12s the patch passed
+1 💚 javac 2m 12s the patch passed
+1 💚 shadedjars 6m 20s patch has no errors when building our shaded downstream artifacts.
+1 💚 javadoc 3m 46s the patch passed
_ Other Tests _
+1 💚 unit 333m 1s root in the patch passed.
369m 14s
Subsystem Report/Notes
Docker ClientAPI=1.41 ServerAPI=1.41 base: https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3560/1/artifact/yetus-jdk8-hadoop2-check/output/Dockerfile
GITHUB PR #3560
Optional Tests javac javadoc unit shadedjars compile
uname Linux cff831f978bd 4.15.0-112-generic #113-Ubuntu SMP Thu Jul 9 23:41:39 UTC 2020 x86_64 x86_64 x86_64 GNU/Linux
Build tool maven
Personality dev-support/hbase-personality.sh
git revision branch-2 / 620806e
Default Java AdoptOpenJDK-1.8.0_282-b08
Test Results https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3560/1/testReport/
Max. process+thread count 5044 (vs. ulimit of 12500)
modules C: hbase-protocol-shaded hbase-common hbase-client hbase-server . U: .
Console output https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3560/1/console
versions git=2.17.1 maven=3.6.3
Powered by Apache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants