Skip to content

[branch-2] HBASE-30538 Bump snappy-java to 1.1.10.12 - #8873

Open
nirdosh0110 wants to merge 3 commits into
apache:branch-2from
nirdosh0110:HBASE-30538-branch-2
Open

nirdosh0110 wants to merge 3 commits into
apache:branch-2from
nirdosh0110:HBASE-30538-branch-2

Conversation

@nirdosh0110

Copy link
Copy Markdown
Contributor

Backport of #8870 to branch-2. https://issues.apache.org/jira/browse/HBASE-30538

Bumps snappy-java 1.1.10.9 -> 1.1.10.12 and adds TestHadoopSnappyCodec.

The test is skipped when Hadoop is older than 3.3.1, since Hadoop's SnappyCodec only uses snappy-java from 3.3.1 on (HADOOP-17125) and needs the native library before that. As suggested in #8870 (comment).

Verified locally on branch-2:

  • default (Hadoop 2.10.2, JDK 8): skipped
  • -Dhadoop.profile=3.0 (Hadoop 3.4.3): pass
  • -Dhadoop.profile=3.0 -Dsnappy.version=1.1.10.9: fails with not enough space for output: need 254,857 bytes, but only 0 remaining

🤖 Generated with Claude Code

nirdosh0110 and others added 3 commits November 13, 2026 01:01
snappy-java 1.1.10.9 added an output-space check in
Snappy.compress(ByteBuffer, ByteBuffer) based on remaining(), which
rejects Hadoop SnappyCompressor (output limit(0) before compress) and
breaks writes through Hadoop SnappyCodec. Fixed in 1.1.10.11 (apache#766);
1.1.10.12 also fixes apache#767.

Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
(cherry picked from commit 4a04384)
Fails with snappy-java 1.1.10.9/1.1.10.10 ('not enough space for
output'), passes with 1.1.10.11+.

Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
(cherry picked from commit a2cfc21)
Hadoop's SnappyCodec only uses snappy-java since 3.3.1 (HADOOP-17125).

Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
@Umeshkumar9414 Umeshkumar9414 added the backport This PR is a back port of some issue or issues already committed to master label Nov 12, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport This PR is a back port of some issue or issues already committed to master

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants