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

[DNM] rebase test of 1644 #4421

Closed
wants to merge 159 commits into from
Closed

[DNM] rebase test of 1644 #4421

wants to merge 159 commits into from

Conversation

alindima
Copy link
Contributor

@alindima alindima commented May 9, 2024

No description provided.

@alindima
Draft of RecoveryStrategy based on linked enums
051328a
@alindima
add copright header
166aae5
@alindima
fix clippy
518d8fd
@alindima
Refactor RecoveryStrategy using dynamic dispatch
dae34a5
@alindima
address comments
640c2d5
@alindima
Merge branch 'master' into alindima/refactor-availability-recovery-st…
ac18bab 
…rategies
@alindima
erasure-coding: add algorithm for systematic recovery
2269f76 
Signed-off-by: alindima <[email protected]>
@alindima
WIP
dca9ce4
@alindima
Merge remote-tracking branch 'origin/master' into alindima/add-system…
9eeb93a 
…atic-chunks-av-recovery
@alindima
continue implementation
6f58c3a
@alindima
Fix some tests
a805a85
@alindima
Merge remote-tracking branch 'origin/master' into alindima/add-system…
d83bf50 
…atic-chunks-av-recovery
@alindima
Merge remote-tracking branch 'origin/master' into alindima/refactor-a…
2a5d6d5 
…vailability-recovery-strategies
@alindima
replace hashmap of chunks with btreemap
f3d407b 
this avoids sorting the chunks for systematic recovery,
which is the hotpath we aim to optimise
@alindima
add chunk indices cache to av-recovery
67841bd
@alindima
some more improvements
3810eab
@alindima
don't use the backing group if chunk size query failed
bf39ba0
@alindima
add ImmediateError to chunks recovery strategy
34408b5
@alindima
modify and add metrics
b68e671
@alindima
more review comments
1569e49
@alindima
fix test
d5c32d1
@alindima
rollback to using TryConnect for fetch chunks
cb42bef
@alindima
add runtime API knob for shuffling enablement
06f00a2
@alindima
av-recovery and av-distr: use relay parent from candidate_receipt
2b58895 
when querying the runtime api
@alindima
add avail_chunk_shuffling params to HostConfiguration
f860b7d
@alindima
replace BypassAvStore variant with a separate flag
b82b2a0
@alindima
move requesting_chunks to the strategy
8585628
@alindima
Merge remote-tracking branch 'origin/master' into alindima/refactor-a…
2f9b767 
…vailability-recovery-strategies
@alindima
Merge remote-tracking branch 'origin/alindima/refactor-availability-r…
824f7e3 
…ecovery-strategies' into alindima/add-systematic-chunks-av-recovery
@alindima
fix tests
818d9c0
alindima and others added 23 commits February 9, 2024 13:15
@alindima
address some review comments
826208f
@alindima
try fixing zombienet tests
d25cc1c
@alindima
Merge remote-tracking branch 'origin/master' into alindima/add-system…
4e958ba 
…atic-chunks-av-recovery
@alindima
more zombienet
dbc27b3
@alindima
zombienet
6aa3b16
@alindima
Merge remote-tracking branch 'origin/master' into alindima/add-system…
bf93b63 
…atic-chunks-av-recovery
@alindima
fix yaml formatting
2316316
@alindima
Merge remote-tracking branch 'origin/master' into alindima/add-system…
0c57295 
…atic-chunks-av-recovery
@alindima
fix import
df8a096
@alindima
fix bench
486cf64
@alindima
clippy
70e8840
@alindima
fix image name
ee93f68
@alindima
Merge remote-tracking branch 'origin/master' into alindima/add-system…
8a2931c 
…atic-chunks-av-recovery
@alindima
try another image for the zombienet test
7d9e21a
@pepoviola
Use cumulus image for collator
9000f5c
@alindima
Merge remote-tracking branch 'origin/master' into alindima/add-system…
e2a95c2 
…atic-chunks-av-recovery
@alindima
fix yaml format
eeab22a
@alindima
Merge remote-tracking branch 'origin/master' into alindima/add-system…
e39ea78 
…atic-chunks-av-recovery
@alindima
fix merge commit
a57f5fd
@alindima
fix bench
6948ea1
@alindima
Merge remote-tracking branch 'origin/master' into alindima/add-system…
91a6180 
…atic-chunks-av-recovery
@alindima
re-add enable-node-feature script
da66e88
@alindima
fix zombienet tests
d20ced9
@alindima alindima requested a review from a team as a code owner May 9, 2024 12:35
@alindima alindima marked this pull request as draft May 9, 2024 12:35
@alindima alindima force-pushed the alindima/add-systematic-chunks-av-recovery-rebased branch from 56e042a to 78fef06 Compare May 10, 2024 06:57
@paritytech-cicd-pr
Copy link

The CI pipeline was cancelled due to failure one of the required jobs.
Job name: cargo-clippy
Logs: https://gitlab.parity.io/parity/mirrors/polkadot-sdk/-/jobs/6169071

@alindima alindima force-pushed the alindima/add-systematic-chunks-av-recovery-rebased branch from 78fef06 to 0cbad5a Compare May 10, 2024 07:13
@alindima alindima closed this May 10, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@alindima @paritytech-cicd-pr @pepoviola