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
Add "query_global_state" query RPC to node #1645
Add "query_global_state" query RPC to node #1645
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM 🎊
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I found one small nitpick
# Conflicts: # client/src/main.rs # client/src/query_state.rs
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There should be a casper_query_global_state()
function added to the FFI, and casper_get_item
should have the #[deprecated]
attribute applied similar to the one suggested for get_item()
.
Also, there should be test cases for this new functionality added to integration_test.rs.
bors r+ |
Build failed: |
bors try |
tryBuild succeeded: |
bors r+ |
Build succeeded: |
CHANGELOG:
BlockHash
orstate_root_hash
param to make query toGlobalState
GlobalStateIdentifier
enum to the rpcs to support the queryquery-global-state
command the rust client that takes in a hash which is identified as either ablock
hash orstate
root hash along with akey
andpath
arg (similar to the preexisting) queryPS: We could use a better distinction between the "state_get_item" query and the newly added with one this PR on the client side. I am open to suggestions for names that are more distinct
Closes: 1608