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
feat(drive-abci): support consensus params updates #1713
base: v1.0-dev
Are you sure you want to change the base?
Conversation
…o feat/drive/read-only-thread
Handle query and check tx in blocked thread pool. Spawn an independent thread with max priority for consensus
9de809c
to
b3e2991
Compare
/// * `Err(io::Error)` - If there was an error reading the file | ||
/// | ||
// TODO: Move this to correct place | ||
pub(super) fn get_consensus_params_update( |
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.
This needs to be versioned, and it's own file.
consensus_param_updates: get_consensus_params_update( | ||
&app.platform().config.abci.consensus_params_dir, | ||
request.height, | ||
) | ||
.map_err(|e| Error::Abci(AbciError::ConsensusParams(e.to_string())))?, |
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.
This can not work, because it would require the entire network to update on the same block. Something that is possible for a devnet, but not in a real network. Consensus params imo should be versioned, and can only change if the network votes for it in consensus.
…and refactor/fix query tests
…o feat/drive/read-only-thread # Conflicts: # packages/rs-drive-abci/src/server.rs
736d1d8
to
7167d4e
Compare
…sensus-params-updates
Issue being fixed or feature implemented
For tuning of ABCI consensus params, we need a way to specify consensus params updates in real time, without modifying or even restarting the Drive.
What was done?
Added an option to propose consensus params updates, based on content of some directory on disk.
Added option consensus_params_dir to ABCI configuration.
TODO: Support in ProcessProposal.
How Has This Been Tested?
Not tested yet.
Breaking Changes
Checklist:
For repository code-owners and collaborators only