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

improve the bonds query #43

Closed
tzemanovic opened this issue Dec 20, 2021 · 3 comments · Fixed by #740
Closed

improve the bonds query #43

tzemanovic opened this issue Dec 20, 2021 · 3 comments · Fixed by #740
Assignees
Labels

Comments

@tzemanovic
Copy link
Member

tzemanovic commented Dec 20, 2021

The anomac bonds --validator <validator-alias> command without the --owner argument doesn't include delegations, it only shows self-bonds. To include delegations we'll need to use a prefix iterator query to find them.

┆Issue is synchronized with this Asana task by Unito

@tzemanovic tzemanovic added cli client enhancement New feature or request labels Dec 20, 2021
@sync-by-unito sync-by-unito bot closed this as completed Feb 3, 2022
@juped juped reopened this Feb 3, 2022
@tzemanovic tzemanovic transferred this issue from anoma/anoma-archive Jul 7, 2022
@cwgoes
Copy link
Contributor

cwgoes commented Jan 12, 2023

@brentstone can you check on this / take this on as part of the PoS work?

@brentstone
Copy link
Contributor

@brentstone can you check on this / take this on as part of the PoS work?

Yeah, I noticed this recently and didn't realize there was an open issue for it. Sounds good.

@brentstone
Copy link
Contributor

Perhaps use a lazy hash set or similar to store all addresses that have delegations to a validator (including self), then querying all delegations becomes much simpler.

@juped juped closed this as completed in #740 Feb 7, 2023
phy-chain pushed a commit to phy-chain/namada that referenced this issue Mar 1, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
No open projects
Status: Tested in Devnet
Development

Successfully merging a pull request may close this issue.

4 participants