feat: implement stake snapshot local state query #394
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
- add a new function `get_stake_snapshots` in the `localstate` queries to make queries about stake snapshots. - add display details to `minicbor` encoding/decoding process in `utils.rs` to aid in debugging. - modify `blockquery` enum to accommodate stake screenshot queries. - rename variable `x` to `pools` in `localstate` queries to reflect its intended usage clearly. - add new types `pools`, `stakesnapshot`, `snapshots`, `stakes` to signify the snapshot of staking state. - the `pools`, `stakesnapshot`, `snapshots`, `stakes` are now encoded and decoded using minicbor in the `queries_v16/mod.rs`. - `get_stake_snapshots` function now returns `stakesnapshot` which has detailed snapshot information of stake pools. - implement a change in the `do_localstate_query` function where a new query of stake snapshots is fired and the result is logged.
- add the ability to query stake snapshots for specified stake pools in the `get_stake_snapshots` function - remove unnecessary code in the `anycbor` struct - remove unnecessary code in the `utils.rs` file - fix a typo in the `queries_v16/mod.rs` file - update the `pallas` dependency to fix a bug in the `n2c-miniprotocols/src/main.rs` file
- change the way the `pool_idx` is initialized in `do_localstate_query` function - add `bytes` import to `protocols.rs` file in the `pallas-network/tests` directory - add a new query to the server's receive query statements in `local_state_query_server_and_client_happy_path` function - add assertions for the new query in `local_state_query_server_and_client_happy_path` function - add stake snapshots and result sending statements to the server's send result statements in `local_state_query_server_and_client_happy_path` function - add sending a query and receiving the result statements to the client's `local_state_query_server_and_client_happy_path` function
- removed code related to stake pool id/verification key hash - changed the `pools` variable to an empty vector
- change the comment to clarify that the stake pool id/verification key hash can be either bech32-decoded or hex-decoded
- modify the comment to reflect that an empty vector means all pools
- remove unused import of `codec::utils::bytes`
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
In case we search for specific pools: currently we have one pool per request limit. Empty pools means all pools.
the
cardano-clihas the same behavior due to this limitation.fixes #395