Skip to content
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

Allow block_processor variables tuning with launch flags #1842

Merged
merged 9 commits into from
Mar 21, 2019

Conversation

SergiySW
Copy link
Contributor

--block_processor_batch_size
--block_processor_full_size
--block_processor_verification_size

Can be useful for fast nodes and database profiling
#1660

--block_processor_batch_size
--block_processor_full_size
--block_processor_verification_size

Can be useful for fast nodes and database profiling
nanocurrency#1660
@SergiySW SergiySW added this to the V19.0 milestone Mar 18, 2019
@SergiySW SergiySW self-assigned this Mar 18, 2019
@SergiySW SergiySW requested review from wezrule and cryptocode March 18, 2019 23:17
Copy link
Contributor

@cryptocode cryptocode left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM pending comment about the CLI description

@SergiySW SergiySW merged commit b47c275 into nanocurrency:master Mar 21, 2019
@zhyatt zhyatt added the documentation This item indicates the need for or supplies updated or expanded documentation label Mar 21, 2019
guilhermelawless pushed a commit to guilhermelawless/nano-node that referenced this pull request Apr 15, 2019
…y#1842)

* Allow block_processor variables tuning with launch flags

--block_processor_batch_size
--block_processor_full_size
--block_processor_verification_size

Can be useful for fast nodes and database profiling
nanocurrency#1660

* Define fast_bootstrap values

* Apply changes from reviews

* Update --debug_profile_bootstrap flags

* Make separate function update_flags

* Removing default node_flags constructor

* Formatting

* Typo
@zhyatt zhyatt removed the documentation This item indicates the need for or supplies updated or expanded documentation label Jul 11, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants