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

Storage Replication refactoring #937

Open
J-Loudet opened this issue Apr 16, 2024 · 0 comments
Open

Storage Replication refactoring #937

J-Loudet opened this issue Apr 16, 2024 · 0 comments
Assignees
Labels
enhancement Existing things could work better

Comments

@J-Loudet
Copy link
Contributor

Describe the feature

As described in #886 the behavior of the Aligner Queryable is not particularly optimized, to quote the changes made in the pull request:

  1. Incrementally replies with digest replies in order to avoid queries timing out w/o sending any data, especially in the case of very outdated storages.
  2. Replace the unbounded queue for digests with a bounded queue to prevent infinitely growing digest queues when digest updates take a long time.

Similarly, #884 removes quite a few unwrap() that have their invariant violated.

This strongly indicates that a deep refactoring of the Storage Replication is needed.

This issue will serve as a central tracking point for Replication-related items.

@J-Loudet J-Loudet added the enhancement Existing things could work better label Apr 16, 2024
@J-Loudet J-Loudet self-assigned this Apr 16, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement Existing things could work better
Projects
None yet
Development

No branches or pull requests

1 participant