Skip to content

HBASE-28655 IllegalArgumentException: Illegal bufferSize thrown when hbase.io.compress.zstd.buffersize is not configured - #6114

Merged
pankaj72981 merged 1 commit into
apache:masterfrom
pankaj72981:HBASE-28655
Jul 25, 2024
Merged

HBASE-28655 IllegalArgumentException: Illegal bufferSize thrown when hbase.io.compress.zstd.buffersize is not configured#6114
pankaj72981 merged 1 commit into
apache:masterfrom
pankaj72981:HBASE-28655

Conversation

@pankaj72981

Copy link
Copy Markdown
Contributor

HADOOP-18810 added io.compression.codec.zstd.buffersize in core-default.xml with default value as 0, so default value of hbase.io.compress.zstd.buffersize will become 0 instead of ZSTD_BUFFER_SIZE_DEFAULT (256 * 1024) when not configured in hbase-site.xml.

…hbase.io.compress.zstd.buffersize is not configured
@Apache-HBase

Copy link
Copy Markdown

🎊 +1 overall

VoteSubsystemRuntimeLogfileComment
+0 🆗reexec0m 35sDocker mode activated.
_ Prechecks _
+1 💚dupname0m 0sNo case conflicting files found.
+0 🆗codespell0m 0scodespell was not available.
+0 🆗detsecrets0m 0sdetect-secrets was not available.
+1 💚@author0m 0sThe patch does not contain any @author tags.
+1 💚hbaseanti0m 0sPatch does not have any anti-patterns.
_ master Compile Tests _
+0 🆗mvndep0m 10sMaven dependency ordering for branch
+1 💚mvninstall3m 4smaster passed
+1 💚compile0m 43smaster passed
+1 💚checkstyle0m 18smaster passed
+1 💚spotbugs0m 47smaster passed
+1 💚spotless0m 45sbranch has no errors when running spotless:check.
_ Patch Compile Tests _
+0 🆗mvndep0m 11sMaven dependency ordering for patch
+1 💚mvninstall2m 56sthe patch passed
+1 💚compile0m 41sthe patch passed
+1 💚javac0m 41sthe patch passed
+1 💚blanks0m 0sThe patch has no blanks issues.
+1 💚checkstyle0m 18sthe patch passed
+1 💚spotbugs0m 58sthe patch passed
+1 💚hadoopcheck10m 43sPatch does not cause any errors with Hadoop 3.3.6 3.4.0.
+1 💚spotless0m 43spatch has no errors when running spotless:check.
_ Other Tests _
+1 💚asflicense0m 20sThe patch does not generate ASF License warnings.
29m 58s
SubsystemReport/Notes
DockerClientAPI=1.46 ServerAPI=1.46 base: https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-6114/1/artifact/yetus-general-check/output/Dockerfile
GITHUB PR#6114
Optional Testsdupname asflicense javac spotbugs checkstyle codespell detsecrets compile hadoopcheck hbaseanti spotless
unameLinux e406412b7e32 5.4.0-182-generic #202-Ubuntu SMP Fri Apr 26 12:29:36 UTC 2024 x86_64 x86_64 x86_64 GNU/Linux
Build toolmaven
Personalitydev-support/hbase-personality.sh
git revisionmaster / 21ebd2a
Default JavaEclipse Adoptium-17.0.11+9
Max. process+thread count81 (vs. ulimit of 30000)
modulesC: hbase-compression/hbase-compression-aircompressor hbase-compression/hbase-compression-zstd U: hbase-compression
Console outputhttps://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-6114/1/console
versionsgit=2.34.1 maven=3.9.8 spotbugs=4.7.3
Powered byApache Yetus 0.15.0 https://yetus.apache.org

This message was automatically generated.

@Apache-HBase

Copy link
Copy Markdown

🎊 +1 overall

VoteSubsystemRuntimeLogfileComment
+0 🆗reexec0m 15sDocker mode activated.
-0 ⚠️yetus0m 3sUnprocessed flag(s): --brief-report-file --spotbugs-strict-precheck --author-ignore-list --blanks-eol-ignore-file --blanks-tabs-ignore-file --quick-hadoopcheck
_ Prechecks _
_ master Compile Tests _
+0 🆗mvndep0m 11sMaven dependency ordering for branch
+1 💚mvninstall4m 57smaster passed
+1 💚compile1m 0smaster passed
+1 💚javadoc0m 51smaster passed
+1 💚shadedjars9m 12sbranch has no errors when building our shaded downstream artifacts.
_ Patch Compile Tests _
+0 🆗mvndep0m 17sMaven dependency ordering for patch
+1 💚mvninstall4m 15sthe patch passed
+1 💚compile0m 39sthe patch passed
+1 💚javac0m 39sthe patch passed
+1 💚javadoc0m 26sthe patch passed
+1 💚shadedjars7m 31spatch has no errors when building our shaded downstream artifacts.
_ Other Tests _
+1 💚unit1m 55shbase-compression-aircompressor in the patch passed.
+1 💚unit6m 23shbase-compression-zstd in the patch passed.
39m 15s
SubsystemReport/Notes
DockerClientAPI=1.46 ServerAPI=1.46 base: https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-6114/1/artifact/yetus-jdk17-hadoop3-check/output/Dockerfile
GITHUB PR#6114
Optional Testsjavac javadoc unit compile shadedjars
unameLinux 559583c71ddd 5.4.0-182-generic #202-Ubuntu SMP Fri Apr 26 12:29:36 UTC 2024 x86_64 x86_64 x86_64 GNU/Linux
Build toolmaven
Personalitydev-support/hbase-personality.sh
git revisionmaster / 21ebd2a
Default JavaEclipse Adoptium-17.0.11+9
Test Resultshttps://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-6114/1/testReport/
Max. process+thread count887 (vs. ulimit of 30000)
modulesC: hbase-compression/hbase-compression-aircompressor hbase-compression/hbase-compression-zstd U: hbase-compression
Console outputhttps://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-6114/1/console
versionsgit=2.34.1 maven=3.9.8
Powered byApache Yetus 0.15.0 https://yetus.apache.org

This message was automatically generated.

@vineet4008vineet4008 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@pankaj72981
pankaj72981 merged commit 73f2710 into apache:masterJul 25, 2024
pankaj72981 added a commit that referenced this pull request Jul 25, 2024
…hbase.io.compress.zstd.buffersize is not configured (#6114)
Signed-off-by: Duo Zhang <zhangduo@apache.org>
Reviewed-by: Vineet Kumar Maheshwari <vineet.4008@gmail.com>
(cherry picked from commit 73f2710)
pankaj72981 added a commit to pankaj72981/hbase that referenced this pull request Jul 25, 2024
…hbase.io.compress.zstd.buffersize is not configured (apache#6114)
Signed-off-by: Duo Zhang <zhangduo@apache.org>
Reviewed-by: Vineet Kumar Maheshwari <vineet.4008@gmail.com>
(cherry picked from commit 73f2710)
pankaj72981 added a commit that referenced this pull request Jul 25, 2024
…hbase.io.compress.zstd.buffersize is not configured (#6114) (#6121)
Signed-off-by: Duo Zhang <zhangduo@apache.org>
Reviewed-by: Vineet Kumar Maheshwari <vineet.4008@gmail.com>
(cherry picked from commit 73f2710)
pankaj72981 added a commit that referenced this pull request Jul 25, 2024
…hbase.io.compress.zstd.buffersize is not configured (#6114) (#6121)
Signed-off-by: Duo Zhang <zhangduo@apache.org>
Reviewed-by: Vineet Kumar Maheshwari <vineet.4008@gmail.com>
(cherry picked from commit 73f2710)
(cherry picked from commit c6e819e)
pankaj72981 added a commit that referenced this pull request Jul 25, 2024
…hbase.io.compress.zstd.buffersize is not configured (#6114) (#6121)
Signed-off-by: Duo Zhang <zhangduo@apache.org>
Reviewed-by: Vineet Kumar Maheshwari <vineet.4008@gmail.com>
(cherry picked from commit 73f2710)
(cherry picked from commit c6e819e)
sanjeet006py pushed a commit to sanjeet006py/hbase that referenced this pull request Aug 24, 2025
…hbase.io.compress.zstd.buffersize is not configured (apache#6114) (apache#6121)
Signed-off-by: Duo Zhang <zhangduo@apache.org>
Reviewed-by: Vineet Kumar Maheshwari <vineet.4008@gmail.com>
(cherry picked from commit 73f2710)
(cherry picked from commit c6e819e)
Signed-off-by: apoonia <apoonia@salesforce.com>
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.

4 participants

@pankaj72981@Apache-HBase@Apache9@vineet4008