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): skip concurrent writes on edsstore Put #2613

Merged
merged 9 commits into from
Sep 6, 2023

Conversation

walldiss
Copy link
Member

@walldiss walldiss commented Aug 25, 2023

Short circuit parallel operations

Closes #2608

@Wondertan
Copy link
Member

@walldiss, wrong issue attached

share/eds/store.go Outdated Show resolved Hide resolved
share/eds/store.go Outdated Show resolved Hide resolved
@codecov-commenter
Copy link

codecov-commenter commented Aug 25, 2023

Codecov Report

Merging #2613 (81eb428) into main (456d169) will increase coverage by 0.15%.
Report is 1 commits behind head on main.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main    #2613      +/-   ##
==========================================
+ Coverage   50.97%   51.13%   +0.15%     
==========================================
  Files         158      158              
  Lines       10638    10642       +4     
==========================================
+ Hits         5423     5442      +19     
+ Misses       4738     4721      -17     
- Partials      477      479       +2     
Files Changed Coverage Δ
share/eds/store.go 69.05% <100.00%> (+5.30%) ⬆️

... and 5 files with indirect coverage changes

@walldiss walldiss marked this pull request as draft August 25, 2023 13:13
@walldiss walldiss marked this pull request as ready for review August 29, 2023 09:10
distractedm1nd
distractedm1nd previously approved these changes Aug 29, 2023
Wondertan
Wondertan previously approved these changes Sep 5, 2023
distractedm1nd
distractedm1nd previously approved these changes Sep 5, 2023
@distractedm1nd
Copy link
Collaborator

linter seems to be failing from the weird cache issue again?

@Wondertan Wondertan enabled auto-merge (squash) September 5, 2023 17:17
@Wondertan Wondertan merged commit 1c84b04 into celestiaorg:main Sep 6, 2023
13 of 15 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area:storage kind:fix Attached to bug-fixing PRs
Projects
None yet
Development

Successfully merging this pull request may close these issues.

fix(eds/store): Synchronize WriteEDS calls
4 participants