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

feat(redundancy/getter): RS decoder rewrite #4507

Merged
merged 1 commit into from
Dec 17, 2023

Conversation

zelig
Copy link
Member

@zelig zelig commented Dec 13, 2023

rewrite of the file/redunancy/getter package with extensive tests
the getter now solicits initial strategies [NONE, DATA, PROX, RACE] with strict or fallback modes.
A third parameter sets the timeout for individual chunk requests that when set below the maximum latency can efffectively simulate erasures.
The API needs to support these as headers in a coming PR.

@zelig zelig self-assigned this Dec 13, 2023
@zelig zelig requested a review from nugaon December 13, 2023 14:41
Copy link
Member

@nugaon nugaon left a comment

Choose a reason for hiding this comment

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

CI cannot build the code.
please reassign review when the test pass

@nugaon nugaon force-pushed the feat/replicas-integration branch from a4196fc to 98777fd Compare December 14, 2023 21:15
This was referenced Dec 14, 2023
Base automatically changed from feat/replicas-integration to feat/redundancy December 14, 2023 22:08
@zelig zelig force-pushed the feat/redundancy-getter branch from 83d4ecf to 39a80f5 Compare December 15, 2023 18:09
This was referenced Dec 16, 2023
@zelig zelig force-pushed the feat/redundancy-getter branch 15 times, most recently from 6797e94 to 780d272 Compare December 17, 2023 21:07
@zelig zelig force-pushed the feat/redundancy-getter branch from 780d272 to 6970486 Compare December 17, 2023 22:21
@zelig zelig merged commit 49f6f2f into feat/redundancy Dec 17, 2023
12 checks passed
@zelig zelig deleted the feat/redundancy-getter branch December 17, 2023 22:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants