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

Remove the lock mechanism in RocksDB #780

Closed
Tracked by #407
jerryshao opened this issue Nov 20, 2023 · 0 comments · Fixed by #2311
Closed
Tracked by #407

Remove the lock mechanism in RocksDB #780

jerryshao opened this issue Nov 20, 2023 · 0 comments · Fixed by #2311
Assignees
@qqqttt123
Milestone
Gravitino 0.5.0

Comments

@jerryshao
Copy link
Contributor

No description provided.

@jerryshao jerryshao mentioned this issue Nov 20, 2023
Closed
4 tasks
@jerryshao jerryshao added this to the Gravitino 0.4.0 milestone Nov 23, 2023
@jerryshao jerryshao removed this from the Gravitino 0.4.0 milestone Jan 23, 2024
@jerryshao jerryshao added this to the Gravitino 0.5.0 milestone Feb 4, 2024
This was referenced Feb 19, 2024
Merged
Merged
jerryshao pushed a commit that referenced this issue Feb 22, 2024
@yuqi1129
[#780] improvment(core): Remove RW lock in KvEntityStore as #779 has …
15e8a4e 
…guaranteed thread safe. (#2311)

### What changes were proposed in this pull request?

Remove the RW lock that guard concurrent issues in `KvEntityStore`.

### Why are the changes needed?

#779 has introduced tree lock to guarantee thread-safe, so we can remove
it.

Fix: #780 

### Does this PR introduce _any_ user-facing change?

N/A.

### How was this patch tested?

Existing tests can cover it.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@qqqttt123 qqqttt123

Labels
None yet
Projects
None yet
Milestone
Gravitino 0.5.0
2 participants
@jerryshao @qqqttt123