Skip to content
This repository has been archived by the owner on Nov 15, 2023. It is now read-only.

Prefer fetching small PoVs from backing group #7173

Merged
merged 7 commits into from
May 5, 2023
Merged

Prefer fetching small PoVs from backing group #7173

merged 7 commits into from
May 5, 2023

Conversation

sandreim
Copy link
Contributor

@sandreim sandreim commented May 3, 2023

Fixes #6832

@sandreim sandreim added A3-in_progress Pull request is in progress. No review needed at this stage. B0-silent Changes should not be mentioned in any release notes C1-low PR touches the given topic and has a low impact on builders. D3-trivial 🧸 PR contains trivial changes in a runtime directory that do not require an audit. labels May 3, 2023
eskimor
eskimor approved these changes May 4, 2023
View reviewed changes
Copy link
Member

@eskimor eskimor left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice!

node/network/availability-recovery/src/lib.rs Outdated Show resolved Hide resolved
node/network/availability-recovery/src/lib.rs Outdated Show resolved Hide resolved
node/network/availability-recovery/src/tests.rs Show resolved Hide resolved
@sandreim sandreim marked this pull request as ready for review May 4, 2023 12:44
@sandreim sandreim requested a review from skunert May 4, 2023 13:16
@sandreim sandreim added A0-please_review Pull request needs code review. and removed A3-in_progress Pull request is in progress. No review needed at this stage. labels May 4, 2023
@sandreim
review feedback
4052bce 
Signed-off-by: Andrei Sandu <[email protected]>
@sandreim sandreim requested a review from ordian May 5, 2023 09:16
@sandreim
Copy link
Contributor Author

sandreim commented May 5, 2023

bot merge

@paritytech-processbot
Copy link

Waiting for commit status.

@paritytech-processbot paritytech-processbot bot merged commit f046636 into master May 5, 2023
@paritytech-processbot paritytech-processbot bot deleted the sandreim/smarter_av_recovery branch May 5, 2023 09:56
@kiltbot kiltbot mentioned this pull request Jun 21, 2023
Closed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@ordian ordian ordian approved these changes

@eskimor eskimor eskimor approved these changes

@skunert skunert Awaiting requested review from skunert

Assignees
No one assigned
Labels
A0-please_review Pull request needs code review. B0-silent Changes should not be mentioned in any release notes C1-low PR touches the given topic and has a low impact on builders. D3-trivial 🧸 PR contains trivial changes in a runtime directory that do not require an audit.
Projects
No open projects
Road to 1k paravalidators and 200 parachains
Status: Done
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

availability-recovery: prefer fetching small PoVs from backing group
4 participants
@sandreim @eskimor @ordian @skunert