Skip to content

HBASE-24587 hbck2 command should accept one or more files containing arguments - #104

Closed
clarax wants to merge 5 commits into
apache:masterfrom
clarax:HBASE-24587-ux
Closed

HBASE-24587 hbck2 command should accept one or more files containing arguments#104
clarax wants to merge 5 commits into
apache:masterfrom
clarax:HBASE-24587-ux

Conversation

@clarax

Copy link
Copy Markdown
Contributor

This is to provide conventional ux for -I option alternative to #69

@Apache-HBase

Copy link
Copy Markdown

💔 -1 overall

VoteSubsystemRuntimeComment
+0 🆗reexec2m 26sDocker mode activated.
_ Prechecks _
+1 💚dupname0m 0sNo case conflicting files found.
+0 🆗markdownlint0m 0smarkdownlint was not available.
+0 🆗spotbugs0m 0sspotbugs executables are not available.
+1 💚@author0m 0sThe patch does not contain any @author tags.
+1 💚test4tests0m 0sThe patch appears to include 2 new or modified test files.
_ master Compile Tests _
+1 💚mvninstall1m 11smaster passed
+1 💚compile0m 18smaster passed
+1 💚checkstyle0m 13smaster passed
+1 💚javadoc0m 12smaster passed
-0 ⚠️patch0m 29sUsed diff version of patch file. Binary files and potentially other changes not applied. Please rebase and squash commits if necessary.
_ Patch Compile Tests _
+1 💚mvninstall0m 21sthe patch passed
+1 💚compile0m 18sthe patch passed
+1 💚javac0m 18sthe patch passed
-1 ❌checkstyle0m 9shbase-hbck2: The patch generated 6 new + 0 unchanged - 0 fixed = 6 total (was 0)
-1 ❌whitespace0m 0sThe patch has 1 line(s) that end in whitespace. Use git apply --whitespace=fix <<patch_file>>. Refer https://git-scm.com/docs/git-apply
+1 💚javadoc0m 8sthe patch passed
_ Other Tests _
-1 ❌unit5m 56shbase-hbck2 in the patch failed.
+1 💚asflicense0m 8sThe patch does not generate ASF License warnings.
11m 36s
ReasonTests
Failed junit testshbase.TestHBCK2
SubsystemReport/Notes
DockerClientAPI=1.41 ServerAPI=1.41 base: https://ci-hbase.apache.org/job/HBase-Operator-Tools-PreCommit/job/PR-104/1/artifact/yetus-precommit-check/output/Dockerfile
GITHUB PR#104
Optional Testsdupname asflicense markdownlint javac javadoc unit spotbugs findbugs checkstyle compile
unameLinux ce365aa77b75 5.4.0-90-generic #101-Ubuntu SMP Fri Oct 15 20:00:55 UTC 2021 x86_64 GNU/Linux
Build toolmaven
git revisionmaster / b7d9fc8
Default JavaOracle Corporation-1.8.0_282-b08
checkstylehttps://ci-hbase.apache.org/job/HBase-Operator-Tools-PreCommit/job/PR-104/1/artifact/yetus-precommit-check/output/diff-checkstyle-hbase-hbck2.txt
whitespacehttps://ci-hbase.apache.org/job/HBase-Operator-Tools-PreCommit/job/PR-104/1/artifact/yetus-precommit-check/output/whitespace-eol.txt
unithttps://ci-hbase.apache.org/job/HBase-Operator-Tools-PreCommit/job/PR-104/1/artifact/yetus-precommit-check/output/patch-unit-hbase-hbck2.txt
Test Resultshttps://ci-hbase.apache.org/job/HBase-Operator-Tools-PreCommit/job/PR-104/1/testReport/
Max. process+thread count1172 (vs. ulimit of 5000)
modulesC: hbase-hbck2 U: hbase-hbck2
Console outputhttps://ci-hbase.apache.org/job/HBase-Operator-Tools-PreCommit/job/PR-104/1/console
versionsgit=2.20.1
Powered byApache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.

@claraxclarax closed this Apr 6, 2022
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.

2 participants

@clarax@Apache-HBase