Skip to content
This repository was archived by the owner on Aug 2, 2022. It is now read-only.

Implement support for read-only contract queries in nodeos#10189

Merged
jgiszczak merged 30 commits intodevelopfrom
read_only_query_EPE-746
May 25, 2021
Merged

Implement support for read-only contract queries in nodeos#10189
jgiszczak merged 30 commits intodevelopfrom
read_only_query_EPE-746

Conversation

@jgiszczak
Copy link
Copy Markdown
Contributor

@jgiszczak jgiszczak commented Mar 26, 2021

Change Description

Implement support for read-only contract queries in nodeos.

Change Type

Select ONE:

  • Documentation
  • Stability bug fix
  • Other
  • Other - special case

Testing Changes

Select ANY that apply:

  • New Tests
  • Existing Tests
  • Test Framework
  • CI System
  • Other

Consensus Changes

  • Consensus Changes

API Changes

  • API Changes

Documentation Additions

  • Documentation Additions

heifner
heifner reviewed Mar 27, 2021
View reviewed changes
Comment thread plugins/producer_plugin/producer_plugin.cpp Outdated
heifner
heifner reviewed Mar 27, 2021
View reviewed changes
Comment thread plugins/producer_plugin/producer_plugin.cpp Outdated
nickjjzhao and others added 16 commits March 31, 2021 20:12
@nickjjzhao
Add resulting URL for read-only queries into cleos
1942d92
@nickjjzhao
Add a transaction in cleos for contract query
ac252a7
@nickjjzhao
Print contract query output as JSON
3ea0af1
@nickjjzhao
Make new cleos command contract_query as a subcommand of get
1fdde7b
@nickjjzhao
Merge pull request #10221 from EOSIO/read_only_query_EPE-747
008b2a3 
Implement cleos support for new RPC API
@jgiszczak
Follow through on new read_only flag in producer plugin
08f3d71
@jgiszczak
Merge branch 'read_only_query_EPE-746' of github.com:/EOSIO/eos into …
a18ec6c 
…read_only_query_EPE-746
@nickjjzhao
Fix a bug that fails to invoke new RPC API get_contract_query
a9b6faf
@nickjjzhao
Merge pull request #10223 from EOSIO/read_only_query_EPE-747
1623b47 
Update cleos support for new RPC API
@nickjjzhao
Use updated structure of get_contract_query_params in cleos
1aa6b10
@nickjjzhao
Merge pull request #10225 from EOSIO/read_only_query_EPE-747
7cf0077 
Use updated structure of get_contract_query_params in cleos
@jgiszczak
Use read_only flag in producer plugin initialize
aafaec5
@jgiszczak
Add read-only query integration test
9bd1d7f
@jgiszczak
Merge develop
836a641
@jgiszczak
Add read-only query integration test contracts
2054d61
@jgiszczak
Merge branch 'develop' into read_only_query_EPE-746
00cb72e
@jgiszczak jgiszczak self-assigned this May 13, 2021
@jgiszczak jgiszczak marked this pull request as ready for review May 13, 2021 14:24
heifner
heifner suggested changes May 14, 2021
View reviewed changes
Comment thread plugins/chain_interface/include/eosio/chain/plugin_interface.hpp Outdated
Comment thread plugins/chain_plugin/chain_plugin.cpp Outdated
Comment thread plugins/chain_plugin/chain_plugin.cpp Outdated
Comment thread plugins/chain_api_plugin/chain_api_plugin.cpp Outdated
Comment thread plugins/chain_plugin/chain_plugin.cpp
Comment thread plugins/producer_plugin/producer_plugin.cpp Outdated
Comment thread plugins/producer_plugin/producer_plugin.cpp Outdated
Comment thread plugins/producer_plugin/producer_plugin.cpp
Comment thread plugins/producer_plugin/producer_plugin.cpp
Comment thread unittests/contracts/CMakeLists.txt
@jgiszczak jgiszczak requested a review from dimas1185 May 17, 2021 15:11
@brianjohnson5972 brianjohnson5972 requested a review from heifner May 20, 2021 21:21
victorj8
victorj8 reviewed May 20, 2021
View reviewed changes
Comment thread unittests/contracts/read_only_query_tests.abi
heifner
heifner reviewed May 24, 2021
View reviewed changes
Comment thread plugins/chain_plugin/include/eosio/chain_plugin/chain_plugin.hpp Outdated
jgiszczak added 2 commits May 24, 2021 11:50
@jgiszczak
Don't return failure traces by default.
df09841
@jgiszczak
Update integration test to push transaction API.
822d90f 
Test verifies new cleos option --return-failure-trace is accepted but
does not test its functionality.
@jgiszczak jgiszczak merged commit f858206 into develop May 25, 2021
@jgiszczak jgiszczak deleted the read_only_query_EPE-746 branch May 25, 2021 06:00
@softprofe softprofe mentioned this pull request Jan 7, 2022
Merged
12 tasks
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Reviewers

@brianjohnson5972 brianjohnson5972 Awaiting requested review from brianjohnson5972

@dimas1185 dimas1185 Awaiting requested review from dimas1185

3 more reviewers

@victorj8 victorj8 victorj8 left review comments

@heifner heifner heifner approved these changes

@larryk85 larryk85 larryk85 approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

@jgiszczak jgiszczak

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

5 participants

@jgiszczak @victorj8 @heifner @larryk85 @nickjjzhao