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

Bump prio-graph version to 0.2.1 #34604

Merged
merged 2 commits into from
Jan 2, 2024
Merged

Conversation

apfitzge
Copy link
Contributor

@apfitzge apfitzge commented Dec 28, 2023

Problem

  • prio-graph 0.2.0 had a bug which did not correctly track write-read conflicts if there were multiple reads after a write. This is fixed in 0.2.1 - see Bugfix: Reads track most recent write apfitzge/prio-graph#57
  • If the write had higher-priority, the scheduler will still have scheduled the write first. However, if they had the same priority, it could have led to an inconsistent scheduling order.

Summary of Changes

  • Bump prio-graph version

Fixes #


Close #34606

@apfitzge apfitzge changed the title num_write_locks_requested Bump prio-graph version to 0.2.1 Dec 28, 2023
Copy link

codecov bot commented Dec 28, 2023

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (5349308) 81.8% compared to head (4b53204) 81.8%.
Report is 2 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff            @@
##           master   #34604     +/-   ##
=========================================
- Coverage    81.8%    81.8%   -0.1%     
=========================================
  Files         824      824             
  Lines      222260   222260             
=========================================
- Hits       181910   181897     -13     
- Misses      40350    40363     +13     

yihau
yihau previously approved these changes Jan 2, 2024
@apfitzge apfitzge added the automerge Merge this Pull Request automatically once CI passes label Jan 2, 2024
@mergify mergify bot merged commit 09a94b8 into solana-labs:master Jan 2, 2024
47 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
automerge Merge this Pull Request automatically once CI passes
Projects
Development

Successfully merging this pull request may close these issues.

2 participants