spark icon indicating copy to clipboard operation
spark copied to clipboard

[SPARK-44223][CORE] Remove LevelDB support

Open Hisoka-X opened this issue 6 months ago • 5 comments

What changes were proposed in this pull request?

This PR remove LevelDB support

Why are the changes needed?

The leveldb project seems to be no longer maintained, and we can always replace it with rocksdb.

Does this PR introduce any user-facing change?

Yes, the LevelDB will not supported in 4.1.0

How was this patch tested?

exist test.

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

No

Hisoka-X avatar May 27 '25 07:05 Hisoka-X

cc @LuciferYang @dongjoon-hyun

Hisoka-X avatar May 27 '25 07:05 Hisoka-X

also cc @mridulm

LuciferYang avatar May 27 '25 08:05 LuciferYang

To @Hisoka-X, perhaps we achieved a limited agreement in the previous pr, but I suggest that it would be better to initiate another relevant discussion on the dev mail list to reach a broader consensus.

LuciferYang avatar May 27 '25 08:05 LuciferYang

To @Hisoka-X, perhaps we achieved a limited agreement in the previous pr, but I suggest that it would be better to initiate another relevant discussion on the dev mail list to reach a broader consensus.

I sent a discussion in dev mail list. https://lists.apache.org/thread/tvf0msobcg1s3v9x14xtbpb7dgvk3zzj

Hisoka-X avatar May 27 '25 08:05 Hisoka-X

@HeartSaVioR has raised a valid concern - what is the path to migration for existing users from leveldb to rocksdb ?

mridulm avatar Jun 02 '25 06:06 mridulm

We're closing this PR because it hasn't been updated in a while. This isn't a judgement on the merit of the PR in any way. It's just a way of keeping the PR queue manageable. If you'd like to revive this PR, please reopen it and ask a committer to remove the Stale tag!

github-actions[bot] avatar Sep 11 '25 00:09 github-actions[bot]