-
Notifications
You must be signed in to change notification settings - Fork 251
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 mempool params to node configuration #193
Conversation
/// Memory pool configuration parameters. | ||
#[derive(Clone, Debug, Serialize, Deserialize)] | ||
pub struct MemoryPoolConfig { | ||
/// Maximum number of uncommited 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.
One more config. 😄
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.
So we have to refactor structure of configurations to better usability.
exonum/src/node/state.rs
Outdated
@@ -298,6 +298,7 @@ impl State { | |||
validator_state: validator_id.map(ValidatorState::new), | |||
public_key: public_key, | |||
secret_key: secret_key, | |||
mempool: mempool, |
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.
Field init shorthand can be used here:
...
secret_key: secret_key,
mempool,
...
@alekseysidorov just in case you're eager to make life easier #34 |
exonum/src/node/mod.rs
Outdated
impl Default for MemoryPoolConfig { | ||
fn default() -> MemoryPoolConfig { | ||
MemoryPoolConfig { | ||
unconfirmed_txs_limit: 20000, |
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.
can you make default value x10 bigger?
28e361d
to
ba6aef5
Compare
ba6aef5
to
83632d2
Compare
Add mempool params to node configuration Former-commit-id: c832fe0ad8d1e3c558386e220420bfa0e9f0c3b4
No description provided.