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

For exists operation it is sufficient to wait for the first response containing true #577

Closed
ikopylov opened this issue Aug 23, 2022 · 1 comment
Closed

For exists operation it is sufficient to wait for the first response containing true #577

ikopylov opened this issue Aug 23, 2022 · 1 comment
Labels
improvement performance

Comments

@ikopylov
Copy link
Member

Currently, exists operation waiting for the answer from all nodes. We can improve performance a little bit by waiting for the first true result.

https://github.com/qoollo/bob/blob/master/bob/src/link_manager.rs#L85
@ikopylov ikopylov assigned lostl1ght and unassigned lostl1ght Oct 12, 2022
@ikopylov
Copy link
Member Author

Fixed in #634

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
improvement performance
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@ikopylov @lostl1ght