-
Notifications
You must be signed in to change notification settings - Fork 1.7k
Conversation
* master: fix compilation warnings (#11522)
@@ -234,7 +234,7 @@ impl<K: Kind, C> VerificationQueue<K, C> { | |||
let number_of_threads = if scale_verifiers { | |||
max_verifiers | |||
} else { | |||
cmp::min(default_amount, max_verifiers) | |||
default_amount |
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 is an unrelated change, sorry
Let's go with this for the time being and see how useful it is. It would be nice to ponder a more general logging overhaul to add more flexibility but this is not the PR for that. |
* master: Code cleanup in the sync module (#11552) initial cleanup (#11542) Warn if genesis constructor revert (#11550) ethcore: cleanup after #11531 (#11546) license update (#11543) Less cloning when importing blocks (#11531) Github Actions (#11528) Fix Alpine Dockerfile (#11538) Remove AuxiliaryData/AuxiliaryRequest (#11533) [journaldb]: cleanup (#11534) Remove references to parity-ethereum (#11525) Drop IPFS support (#11532) chain-supplier: fix warning reporting for GetNodeData request (#11530) Faster kill_garbage (#11514) [EngineSigner]: don't sign message with only zeroes (#11524)
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.
Do you know if/when more fine-grained (caching) would be useful and why not provide all statistics that kvdb
provides?
The code looks good.
I think the general consensus is that maintaining multiple levels of caches is a bad idea and we should delegate caching and reuse it from the low-level db, like rocksdb and fs cache.
Not all stats are implemented for kvdb-rocksdb |
Co-Authored-By: David <[email protected]>
* master: informant: display I/O stats (#11523) [devp2p discovery]: remove `deprecated_echo_hash` (#11564) [secretstore] create db_version file when database doesn't exist (#11570) Remove Parity's Security Policy (#11565) ethcore/res: enable ecip-1088 phoenix upgrade for kotti and mordor testnets (#11529) Misc docs and renames …and one less clone (#11556) [secretstore]: don't sign message with only zeroes (#11561) [devp2p discovery]: cleanup (#11547) Code cleanup in the sync module (#11552) initial cleanup (#11542) Warn if genesis constructor revert (#11550)
Not sure what is the best way to display the logs hence the draft: