Skip to content

Commit

Permalink
[SPARK-47838][BUILD] Upgrade rocksdbjni to 8.11.4
Browse files Browse the repository at this point in the history
### What changes were proposed in this pull request?

Upgrades `rocksdbjni` dependency to 8.11.4.

### Why are the changes needed?

8.11.4 has Java-related RocksDB fixes:

https://github.com/facebook/rocksdb/releases/tag/v8.11.4

- Fixed CMake Javadoc build
- Fixed Java SstFileMetaData to prevent throwing java.lang.NoSuchMethodError

### Does this PR introduce _any_ user-facing change?

No.

### How was this patch tested?

- All existing UTs should pass
- [In progress] Performance benchmarks

### Was this patch authored or co-authored using generative AI tooling?

No.

Closes apache#46065 from neilramaswamy/spark-47838.

Authored-by: Neil Ramaswamy <[email protected]>
Signed-off-by: Dongjoon Hyun <[email protected]>
  • Loading branch information
neilramaswamy authored and dongjoon-hyun committed Apr 17, 2024
1 parent 6c827c1 commit ab6338e
Show file tree
Hide file tree
Showing 4 changed files with 126 additions and 122 deletions.
2 changes: 1 addition & 1 deletion dev/deps/spark-deps-hadoop-3-hive-2.3
Original file line number Diff line number Diff line change
Expand Up @@ -247,7 +247,7 @@ parquet-jackson/1.13.1//parquet-jackson-1.13.1.jar
pickle/1.3//pickle-1.3.jar
py4j/0.10.9.7//py4j-0.10.9.7.jar
remotetea-oncrpc/1.1.2//remotetea-oncrpc-1.1.2.jar
rocksdbjni/8.11.3//rocksdbjni-8.11.3.jar
rocksdbjni/8.11.4//rocksdbjni-8.11.4.jar
scala-collection-compat_2.13/2.7.0//scala-collection-compat_2.13-2.7.0.jar
scala-compiler/2.13.13//scala-compiler-2.13.13.jar
scala-library/2.13.13//scala-library-2.13.13.jar
Expand Down
2 changes: 1 addition & 1 deletion pom.xml
Original file line number Diff line number Diff line change
Expand Up @@ -687,7 +687,7 @@
<dependency>
<groupId>org.rocksdb</groupId>
<artifactId>rocksdbjni</artifactId>
<version>8.11.3</version>
<version>8.11.4</version>
</dependency>
<dependency>
<groupId>${leveldbjni.group}</groupId>
Expand Down
Loading

0 comments on commit ab6338e

Please sign in to comment.