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

fix(share/eds): remove Has() and Sync() badger calls in inverted index #2518

Merged
merged 4 commits into from
Jul 31, 2023

Conversation

walldiss
Copy link
Member

Remove unnecessary Has() and Sync() badger calls in inverted index. Removal of calls greatly improve Put() performance on eds.Store

Wondertan
Wondertan previously approved these changes Jul 28, 2023
renaynay
renaynay previously approved these changes Jul 31, 2023
share/eds/inverted_index.go Show resolved Hide resolved
@walldiss walldiss enabled auto-merge (squash) July 31, 2023 11:49
@walldiss walldiss added the kind:fix Attached to bug-fixing PRs label Jul 31, 2023
@walldiss walldiss dismissed stale reviews from renaynay and Wondertan via 75322b3 July 31, 2023 12:25
@walldiss walldiss merged commit 208bb86 into celestiaorg:main Jul 31, 2023
12 of 14 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area:shares Shares and samples area:storage kind:fix Attached to bug-fixing PRs
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants