spark icon indicating copy to clipboard operation
spark copied to clipboard

[WIP] Only test rocksdbjni 9.x

Open panbingkun opened this issue 10 months ago • 4 comments

What changes were proposed in this pull request?

Why are the changes needed?

Does this PR introduce any user-facing change?

How was this patch tested?

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

panbingkun avatar Apr 20 '24 01:04 panbingkun

At present, we are only testing rocksdbjni's 9 series in advance

panbingkun avatar Apr 20 '24 01:04 panbingkun

Let's test 9.1.1

LuciferYang avatar Apr 23 '24 11:04 LuciferYang

Let's test 9.1.1

Okay. currently, the version 9.1.1 is not available on Maven Central Repo, and the rocksdbjni has been released too frequently recently.

https://repo1.maven.org/maven2/org/rocksdb/rocksdbjni/

panbingkun avatar Apr 23 '24 11:04 panbingkun

We can wait for a relatively stable version.

LuciferYang avatar Apr 23 '24 11:04 LuciferYang

ready to go ?

LuciferYang avatar Jun 24 '24 02:06 LuciferYang

ready to go ?

Yea, the last upgrade was two months ago, Let me run the benchmark: StateStoreBasicOperationsBenchmark JDK17: https://github.com/panbingkun/spark/actions/runs/9640180869 JDK21: https://github.com/panbingkun/spark/actions/runs/9640184179

panbingkun avatar Jun 24 '24 05:06 panbingkun

ready to go ?

Yea, the last upgrade was two months ago, Let me run the benchmark: StateStoreBasicOperationsBenchmark JDK17: https://github.com/panbingkun/spark/actions/runs/9640180869 JDK21: https://github.com/panbingkun/spark/actions/runs/9640184179

Done

panbingkun avatar Jun 24 '24 07:06 panbingkun

Let's wait one more day

LuciferYang avatar Jun 24 '24 14:06 LuciferYang

Merged into master. Thanks @panbingkun

LuciferYang avatar Jun 25 '24 03:06 LuciferYang

Merged into master. Thanks @panbingkun

Thanks.

panbingkun avatar Jun 25 '24 07:06 panbingkun