Skip to content

Conversation

Hisoka-X
Copy link
Member

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
Copy link
Member Author

cc @LuciferYang @dongjoon-hyun

@LuciferYang
Copy link
Contributor

also cc @mridulm

@LuciferYang
Copy link
Contributor

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.

@Hisoka-X
Copy link
Member Author

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

@mridulm
Copy link
Contributor

mridulm commented Jun 2, 2025

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

Copy link

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 github-actions bot added the Stale label Sep 11, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants