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.

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

Support putNoIndex interface on kvstore. #461

Closed
dangleptr opened this issue May 29, 2019 · 4 comments
Closed

Support putNoIndex interface on kvstore. #461

dangleptr opened this issue May 29, 2019 · 4 comments
Milestone

Comments

@dangleptr
Copy link
Contributor

dangleptr commented May 29, 2019

Subtask of #71

When updating data with index, we should read the old data to construct old index-key, then we delete the old index-key, finally insert the new data and new index.

This should be atomic, we could support it in kvstore.

@darionyaphet
Copy link
Contributor

Here are some cost when serialization and deserialization.

Should we save the indexed values in some place to avoid it ?

@darionyaphet
Copy link
Contributor

You mean support read the old data, delete the old index-key, insert the new data and new index in kvstore?

@sherman-the-tank sherman-the-tank changed the title Support putWithIndex interface on kvstore. Support putNoIndex interface on kvstore. May 30, 2019
@sherman-the-tank
Copy link
Member

We need more subtle design on this.

@jude-zhu jude-zhu added this to the R201910_RC1 milestone Aug 7, 2019
@dangleptr
Copy link
Contributor Author

redundant with #458

yixinglu pushed a commit to yixinglu/nebula that referenced this issue Mar 21, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants