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

introduce query policy #475

Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open

introduce query policy #475

wants to merge 2 commits into from

Conversation

adecaro
Copy link
Contributor

@adecaro adecaro commented May 4, 2023

This PR allows the developer to choose a policy to decide how many responses are needed to accept the result of a query. The PR also allows the developer to make use of an external context to track the progress.

@adecaro adecaro self-assigned this May 4, 2023
@adecaro adecaro linked an issue May 6, 2023 that may be closed by this pull request

const (
// QueryAll requires an answer from all selected peers
QueryAll QueryPolicy = iota
Copy link
Contributor

Choose a reason for hiding this comment

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

Maybe we can use the constant defined above

@adecaro adecaro force-pushed the f-474 branch 2 times, most recently from 8cd289e to 4b5b4d1 Compare December 11, 2024 06:49
@adecaro adecaro requested a review from arner December 11, 2024 06:50
adecaro and others added 2 commits December 12, 2024 07:24
Signed-off-by: Angelo De Caro <[email protected]>
Signed-off-by: Angelo De Caro <[email protected]>
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.

Allow successful queries if one of the configured peers is down
2 participants