Skip to content

HBASE-18999 Shell support for multiple columns for DML commands - #5585

Draft
NihalJain wants to merge 2 commits into
apache:masterfrom
NihalJain:HBASE-18999
Draft

HBASE-18999 Shell support for multiple columns for DML commands#5585
NihalJain wants to merge 2 commits into
apache:masterfrom
NihalJain:HBASE-18999

Conversation

@NihalJain

Copy link
Copy Markdown
Contributor

No description provided.

@NihalJain
NihalJain marked this pull request as draft December 16, 2023 15:01
@NihalJain

NihalJain commented Dec 16, 2023

Copy link
Copy Markdown
ContributorAuthor

Draft PR with changes from the last submitted patch in HBASE-18999. I have run hbase build with this change along with the shell test cases.

Pending:

  • Will do a self review and see if code/docs can be improved. For example I see some cryptic code in put.rb /append.rb
  • And also test out using a shell instance.

CC: @Apache9

Comment threadhbase-shell/src/main/ruby/shell/commands/append.rb
Comment threadhbase-shell/src/main/ruby/shell/commands/put.rb
@Apache-HBase

Copy link
Copy Markdown

🎊 +1 overall

VoteSubsystemRuntimeComment
+0 🆗reexec2m 15sDocker mode activated.
_ Prechecks _
+1 💚dupname0m 0sNo case conflicting files found.
+1 💚@author0m 0sThe patch does not contain any @author tags.
_ master Compile Tests _
+1 💚mvninstall3m 5smaster passed
+1 💚spotless0m 45sbranch has no errors when running spotless:check.
_ Patch Compile Tests _
+1 💚mvninstall2m 52sthe patch passed
-0 ⚠️rubocop0m 19sThe patch generated 200 new + 395 unchanged - 12 fixed = 595 total (was 407)
+1 💚whitespace0m 0sThe patch has no whitespace issues.
+1 💚spotless0m 41spatch has no errors when running spotless:check.
_ Other Tests _
+1 💚asflicense0m 10sThe patch does not generate ASF License warnings.
11m 15s
SubsystemReport/Notes
DockerClientAPI=1.43 ServerAPI=1.43 base: https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5585/1/artifact/yetus-general-check/output/Dockerfile
GITHUB PR#5585
Optional Testsdupname asflicense javac spotless rubocop
unameLinux 0cb042f9f2ce 5.4.0-1103-aws #111~18.04.1-Ubuntu SMP Tue May 23 20:04:10 UTC 2023 x86_64 x86_64 x86_64 GNU/Linux
Build toolmaven
Personalitydev-support/hbase-personality.sh
git revisionmaster / b7c3f8c
Default JavaEclipse Adoptium-11.0.17+8
rubocophttps://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5585/1/artifact/yetus-general-check/output/diff-patch-rubocop.txt
Max. process+thread count79 (vs. ulimit of 30000)
modulesC: hbase-shell U: hbase-shell
Console outputhttps://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5585/1/console
versionsgit=2.34.1 maven=3.8.6 rubocop=1.37.1
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 10sDocker 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 _
+1 💚mvninstall2m 59smaster passed
+1 💚javadoc0m 10smaster passed
_ Patch Compile Tests _
+1 💚mvninstall2m 41sthe patch passed
+1 💚javadoc0m 9sthe patch passed
_ Other Tests _
+1 💚unit6m 44shbase-shell in the patch passed.
15m 42s
SubsystemReport/Notes
DockerClientAPI=1.43 ServerAPI=1.43 base: https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5585/1/artifact/yetus-jdk11-hadoop3-check/output/Dockerfile
GITHUB PR#5585
Optional Testsjavac javadoc unit
unameLinux 1a1668a49eb5 5.4.0-1103-aws #111~18.04.1-Ubuntu SMP Tue May 23 20:04:10 UTC 2023 x86_64 x86_64 x86_64 GNU/Linux
Build toolmaven
Personalitydev-support/hbase-personality.sh
git revisionmaster / b7c3f8c
Default JavaEclipse Adoptium-11.0.17+8
Test Resultshttps://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5585/1/testReport/
Max. process+thread count1585 (vs. ulimit of 30000)
modulesC: hbase-shell U: hbase-shell
Console outputhttps://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5585/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.

@Apache-HBase

Copy link
Copy Markdown

🎊 +1 overall

VoteSubsystemRuntimeComment
+0 🆗reexec3m 26sDocker 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 _
+1 💚mvninstall2m 36smaster passed
+1 💚javadoc0m 12smaster passed
_ Patch Compile Tests _
+1 💚mvninstall2m 20sthe patch passed
+1 💚javadoc0m 8sthe patch passed
_ Other Tests _
+1 💚unit6m 53shbase-shell in the patch passed.
16m 28s
SubsystemReport/Notes
DockerClientAPI=1.43 ServerAPI=1.43 base: https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5585/1/artifact/yetus-jdk8-hadoop3-check/output/Dockerfile
GITHUB PR#5585
Optional Testsjavac javadoc unit
unameLinux cc1bc9bd7a55 5.4.0-163-generic #180-Ubuntu SMP Tue Sep 5 13:21:23 UTC 2023 x86_64 x86_64 x86_64 GNU/Linux
Build toolmaven
Personalitydev-support/hbase-personality.sh
git revisionmaster / b7c3f8c
Default JavaTemurin-1.8.0_352-b08
Test Resultshttps://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5585/1/testReport/
Max. process+thread count1525 (vs. ulimit of 30000)
modulesC: hbase-shell U: hbase-shell
Console outputhttps://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5585/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.

@NihalJain

Copy link
Copy Markdown
ContributorAuthor

Pushed local changes, still not ready, will do a little more self review and testing. Will take up later.

@Apache-HBase

Copy link
Copy Markdown

🎊 +1 overall

VoteSubsystemRuntimeComment
+0 🆗reexec0m 25sDocker mode activated.
_ Prechecks _
+1 💚dupname0m 0sNo case conflicting files found.
+1 💚@author0m 0sThe patch does not contain any @author tags.
_ master Compile Tests _
+1 💚mvninstall3m 24smaster passed
+1 💚spotless0m 43sbranch has no errors when running spotless:check.
_ Patch Compile Tests _
+1 💚mvninstall2m 53sthe patch passed
-0 ⚠️rubocop0m 20sThe patch generated 201 new + 396 unchanged - 15 fixed = 597 total (was 411)
+1 💚whitespace0m 0sThe patch has no whitespace issues.
+1 💚spotless0m 41spatch has no errors when running spotless:check.
_ Other Tests _
+1 💚asflicense0m 8sThe patch does not generate ASF License warnings.
9m 46s
SubsystemReport/Notes
DockerClientAPI=1.43 ServerAPI=1.43 base: https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5585/2/artifact/yetus-general-check/output/Dockerfile
GITHUB PR#5585
Optional Testsdupname asflicense javac spotless rubocop
unameLinux 95c7088b2ea7 5.4.0-1103-aws #111~18.04.1-Ubuntu SMP Tue May 23 20:04:10 UTC 2023 x86_64 x86_64 x86_64 GNU/Linux
Build toolmaven
Personalitydev-support/hbase-personality.sh
git revisionmaster / 4aeabdc
Default JavaEclipse Adoptium-11.0.17+8
rubocophttps://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5585/2/artifact/yetus-general-check/output/diff-patch-rubocop.txt
Max. process+thread count79 (vs. ulimit of 30000)
modulesC: hbase-shell U: hbase-shell
Console outputhttps://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5585/2/console
versionsgit=2.34.1 maven=3.8.6 rubocop=1.37.1
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 37sDocker 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 _
+1 💚mvninstall2m 58smaster passed
+1 💚javadoc0m 12smaster passed
_ Patch Compile Tests _
+1 💚mvninstall2m 48sthe patch passed
+1 💚javadoc0m 11sthe patch passed
_ Other Tests _
-1 ❌unit6m 59shbase-shell in the patch failed.
14m 27s
SubsystemReport/Notes
DockerClientAPI=1.43 ServerAPI=1.43 base: https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5585/2/artifact/yetus-jdk11-hadoop3-check/output/Dockerfile
GITHUB PR#5585
Optional Testsjavac javadoc unit
unameLinux 1c62e1db533a 5.4.0-169-generic #187-Ubuntu SMP Thu Nov 23 14:52:28 UTC 2023 x86_64 x86_64 x86_64 GNU/Linux
Build toolmaven
Personalitydev-support/hbase-personality.sh
git revisionmaster / 4aeabdc
Default JavaEclipse Adoptium-11.0.17+8
unithttps://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5585/2/artifact/yetus-jdk11-hadoop3-check/output/patch-unit-hbase-shell.txt
Test Resultshttps://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5585/2/testReport/
Max. process+thread count2029 (vs. ulimit of 30000)
modulesC: hbase-shell U: hbase-shell
Console outputhttps://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5585/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

Copy link
Copy Markdown

💔 -1 overall

VoteSubsystemRuntimeComment
+0 🆗reexec0m 26sDocker mode activated.
-0 ⚠️yetus0m 2sUnprocessed flag(s): --brief-report-file --spotbugs-strict-precheck --whitespace-eol-ignore-list --whitespace-tabs-ignore-list --quick-hadoopcheck
_ Prechecks _
_ master Compile Tests _
+1 💚mvninstall3m 37smaster passed
+1 💚javadoc0m 14smaster passed
_ Patch Compile Tests _
+1 💚mvninstall2m 24sthe patch passed
+1 💚javadoc0m 8sthe patch passed
_ Other Tests _
-1 ❌unit7m 45shbase-shell in the patch failed.
15m 28s
SubsystemReport/Notes
DockerClientAPI=1.43 ServerAPI=1.43 base: https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5585/2/artifact/yetus-jdk8-hadoop3-check/output/Dockerfile
GITHUB PR#5585
Optional Testsjavac javadoc unit
unameLinux b2ef01abe68e 5.4.0-1103-aws #111~18.04.1-Ubuntu SMP Tue May 23 20:04:10 UTC 2023 x86_64 x86_64 x86_64 GNU/Linux
Build toolmaven
Personalitydev-support/hbase-personality.sh
git revisionmaster / 4aeabdc
Default JavaTemurin-1.8.0_352-b08
unithttps://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5585/2/artifact/yetus-jdk8-hadoop3-check/output/patch-unit-hbase-shell.txt
Test Resultshttps://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5585/2/testReport/
Max. process+thread count1987 (vs. ulimit of 30000)
modulesC: hbase-shell U: hbase-shell
Console outputhttps://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5585/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.

@Apache9

Copy link
Copy Markdown
Contributor

Any updates here? @NihalJain

Plan to make the final 3.0.0-beta-2 release, so I'm cleaning up the pending issues for 3.0.0-beta-2.

@NihalJain

Copy link
Copy Markdown
ContributorAuthor

ah sure, let me try start this off again.

@Apache9

Copy link
Copy Markdown
Contributor

@NihalJain Do you still have time to finish this before we cut 3.0.0-beta-2?

There are only 4 issues left for now.

@Apache9

Copy link
Copy Markdown
Contributor

Ping @NihalJain

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

@NihalJain@Apache-HBase@Apache9