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

HBASE-23669 ZNodeClearer#clear() throws IllegalArgumentException #1035

Open
wants to merge 7 commits into
base: branch-2.1
Choose a base branch
from

Conversation

Gkkkk302
Copy link
Contributor

No description provided.

@Gkkkk302
Copy link
Contributor Author

It looks like hbase-14861 is not fully repaired

@Apache-HBase
Copy link

💔 -1 overall

Vote Subsystem Runtime Comment
+0 🆗 reexec 0m 39s Docker mode activated.
_ Prechecks _
+1 💚 dupname 0m 0s No case conflicting files found.
+1 💚 hbaseanti 0m 0s Patch does not have any anti-patterns.
+1 💚 @author 0m 0s The patch does not contain any @author tags.
-0 ⚠️ test4tests 0m 0s 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.
_ branch-2.1 Compile Tests _
+1 💚 mvninstall 5m 16s branch-2.1 passed
+1 💚 compile 0m 55s branch-2.1 passed
+1 💚 checkstyle 1m 18s branch-2.1 passed
+1 💚 shadedjars 4m 0s branch has no errors when building our shaded downstream artifacts.
+1 💚 javadoc 0m 36s branch-2.1 passed
+0 🆗 spotbugs 3m 0s Used deprecated FindBugs config; considering switching to SpotBugs.
+1 💚 findbugs 2m 58s branch-2.1 passed
_ Patch Compile Tests _
+1 💚 mvninstall 5m 4s the patch passed
+1 💚 compile 0m 57s the patch passed
+1 💚 javac 0m 57s the patch passed
-1 ❌ checkstyle 1m 23s hbase-server: The patch generated 1 new + 2 unchanged - 0 fixed = 3 total (was 2)
+1 💚 whitespace 0m 0s The patch has no whitespace issues.
+1 💚 shadedjars 4m 16s patch has no errors when building our shaded downstream artifacts.
+1 💚 hadoopcheck 20m 32s Patch does not cause any errors with Hadoop 2.7.7 2.8.5 or 3.0.3 3.1.2.
+1 💚 javadoc 0m 35s the patch passed
+1 💚 findbugs 3m 2s the patch passed
_ Other Tests _
+1 💚 unit 153m 39s hbase-server in the patch passed.
+1 💚 asflicense 0m 33s The patch does not generate ASF License warnings.
211m 59s
Subsystem Report/Notes
Docker Client=19.03.5 Server=19.03.5 base: https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1035/1/artifact/out/Dockerfile
GITHUB PR #1035
Optional Tests dupname asflicense javac javadoc unit spotbugs findbugs shadedjars hadoopcheck hbaseanti checkstyle compile
uname Linux 79afed3dc797 4.15.0-58-generic #64-Ubuntu SMP Tue Aug 6 11:12:41 UTC 2019 x86_64 GNU/Linux
Build tool maven
Personality /home/jenkins/jenkins-slave/workspace/Base-PreCommit-GitHub-PR_PR-1035/out/precommit/personality/provided.sh
git revision branch-2.1 / f812f61
Default Java 1.8.0_181
checkstyle https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1035/1/artifact/out/diff-checkstyle-hbase-server.txt
Test Results https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1035/1/testReport/
Max. process+thread count 4204 (vs. ulimit of 10000)
modules C: hbase-server U: hbase-server
Console output https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1035/1/console
versions git=2.11.0 maven=2018-06-17T18:33:14Z) findbugs=3.1.11
Powered by Apache Yetus 0.11.1 https://yetus.apache.org

This message was automatically generated.

@Gkkkk302 Gkkkk302 requested a review from busbey January 17, 2020 09:59
@Gkkkk302 Gkkkk302 closed this Apr 9, 2020
@Apache-HBase
Copy link

🎊 +1 overall

Vote Subsystem Runtime Comment
+0 🆗 reexec 0m 45s Docker mode activated.
_ Prechecks _
+1 💚 dupname 0m 0s No case conflicting files found.
+1 💚 hbaseanti 0m 0s Patch does not have any anti-patterns.
+1 💚 @author 0m 0s The patch does not contain any @author tags.
-0 ⚠️ test4tests 0m 0s 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.
_ branch-2.1 Compile Tests _
+1 💚 mvninstall 5m 9s branch-2.1 passed
+1 💚 compile 0m 54s branch-2.1 passed
+1 💚 checkstyle 1m 22s branch-2.1 passed
+1 💚 shadedjars 4m 2s branch has no errors when building our shaded downstream artifacts.
+1 💚 javadoc 0m 36s branch-2.1 passed
+0 🆗 spotbugs 2m 51s Used deprecated FindBugs config; considering switching to SpotBugs.
+1 💚 findbugs 2m 48s branch-2.1 passed
_ Patch Compile Tests _
+1 💚 mvninstall 4m 35s the patch passed
+1 💚 compile 0m 55s the patch passed
+1 💚 javac 0m 55s the patch passed
+1 💚 checkstyle 1m 17s the patch passed
+1 💚 whitespace 0m 0s The patch has no whitespace issues.
+1 💚 shadedjars 4m 3s patch has no errors when building our shaded downstream artifacts.
+1 💚 hadoopcheck 21m 9s Patch does not cause any errors with Hadoop 2.7.7 2.8.5 or 3.0.3 3.1.2.
+1 💚 javadoc 0m 38s the patch passed
+1 💚 findbugs 3m 0s the patch passed
_ Other Tests _
+1 💚 unit 147m 16s hbase-server in the patch passed.
+1 💚 asflicense 0m 33s The patch does not generate ASF License warnings.
205m 23s
Subsystem Report/Notes
Docker Client=19.03.8 Server=19.03.8 base: https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1035/2/artifact/out/Dockerfile
GITHUB PR #1035
Optional Tests dupname asflicense javac javadoc unit spotbugs findbugs shadedjars hadoopcheck hbaseanti checkstyle compile
uname Linux cd9f1e42ffa5 4.15.0-58-generic #64-Ubuntu SMP Tue Aug 6 11:12:41 UTC 2019 x86_64 GNU/Linux
Build tool maven
Personality /home/jenkins/jenkins-slave/workspace/Base-PreCommit-GitHub-PR_PR-1035/out/precommit/personality/provided.sh
git revision branch-2.1 / f2571d4
Default Java 1.8.0_181
Test Results https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1035/2/testReport/
Max. process+thread count 4445 (vs. ulimit of 10000)
modules C: hbase-server U: hbase-server
Console output https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1035/2/console
versions git=2.11.0 maven=2018-06-17T18:33:14Z) findbugs=3.1.11
Powered by Apache Yetus 0.11.1 https://yetus.apache.org

This message was automatically generated.

@Gkkkk302 Gkkkk302 reopened this Jan 6, 2022
@Gkkkk302
Copy link
Contributor Author

Gkkkk302 commented Mar 4, 2022

@busbey Can you take a look at this PR?Thanks

@Gkkkk302
Copy link
Contributor Author

Gkkkk302 commented Mar 4, 2022

@Apache9 Can you take a look at this PR?Thanks

@busbey
Copy link
Contributor

busbey commented Mar 4, 2022

Please start with a change that targets the master branch.

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