Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

storage: deprecate --state-store in-memory #3145

Closed
BugenZhao opened this issue Jun 12, 2022 · 0 comments · Fixed by #3153
Closed

storage: deprecate --state-store in-memory #3145

BugenZhao opened this issue Jun 12, 2022 · 0 comments · Fixed by #3153
Labels
component/storage Storage type/chore user-facing-changes Contains changes that are visible to users

Comments

@BugenZhao
Copy link
Member

After we support more and more features with Hummock, the BTreeMap based in-memory state store is becoming unusable in end-to-end scenarios. We should only allow the test code to rely on this, while removing the entrance for creating with command line arguments in StateStoreImpl::new.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
component/storage Storage type/chore user-facing-changes Contains changes that are visible to users
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

chore(storage): deprecate in-memory state store in e2e scenarios risingwavelabs/risingwave
1 participant
@BugenZhao