Add max open files limit of 512 to rocksdb init #85
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Requires status-im/nim-rocksdb#20
Change rocksdb default of -1 to 512, as we reach the 1024 usual (?) open files limit in Linux when running Nimbus at around block ~1.300.000.
I could not immediately find what the default limit is in Windows, so perhaps I just have to test it.
Setting of 512 is somewhat based on value used in parity client: https://github.com/paritytech/parity-common/blob/master/kvdb-rocksdb/src/lib.rs#L197
Should be quite safe, and if needed can probably be increased safely to somewhere between 700-800. Will need incoming connection limitations for that, but that is another issue.
Could also forward this setting to Nimbus + Nimbus config but decided to not yet do this until it is proven to have an practical impact.
I'll create an issue to benchmark + (if needed) tune rocksdb at later moment.