-
Notifications
You must be signed in to change notification settings - Fork 3.6k
Support read only transactions 📦 #11020
Support read only transactions 📦 #11020
Conversation
…mmand push transaction
Test verifies new cleos option --return-failure-trace is accepted but does not test its functionality.
…8-support-read-only-transactions-develop-boxed
…en launching cluster
} | ||
} | ||
next(read_only::push_ro_transaction_results{db.head_block_num(), db.head_block_id(), db.last_irreversible_block_num(), db.last_irreversible_block_id(), | ||
accnt_metadata_obj.code_hash, pending_transactions, output}); |
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.
Should use std::move(pending_transactions)
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 think you missed this one.
plugins/amqp_trx_plugin/include/eosio/amqp_trx_plugin/fifo_trx_processing_queue.hpp
Outdated
Show resolved
Hide resolved
plugins/chain_plugin/include/eosio/chain_plugin/chain_plugin.hpp
Outdated
Show resolved
Hide resolved
…sactions-develop-boxed
} | ||
next(read_only::push_ro_transaction_results{db.head_block_num(), db.head_block_id(), db.last_irreversible_block_num(), db.last_irreversible_block_id(), | ||
accnt_metadata_obj.code_hash, pending_transactions, output}); |
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.
next
always has to be called. It needs moved outside the if
.
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.
Moved out of if, but compile failed. the parameter of next() need change, just paste the code that you want here to save the forth and back.
Change Description
Support read only transactions develop boxed. See EPE 1898.
Moving PR #10189 to develop-boxed. Solved the conflict and made some adaption for compile problem.
If any more eos PR need for read only query feature, will move in other PR. this PR just for move 10189.
Change Type
Select ONE:
Testing Changes
Select ANY that apply:
Consensus Changes
API Changes
Documentation Additions