yangzhe1990

The average TPS of the storage benchmark from 10M to 100M on my last run at end Feburary is more than 25000. But on master now it seems to be...

Oh, the performance degrade of the benchmark is due to debug validation code here https://github.com/Conflux-Chain/conflux-rust/commit/27fb468481e1008c729f78c17f4aa87fa7bf180a But it wasn't enabled in test actually..

Related work: #1365 should improve the TPS. I wonder wnat's the new TPS number?

I'll be re-assigned for a proper low priority fix.

Checked with @peilun-conflux I think a better fix is to remove all existing snapshots after successful sync, so that the old data don't interfere with the new state.

I'm working on this issue. The temporary fix is to remove the inflight_keys in on_removed(), but when resend() is called, return a GetBlocks request rather than the GetBlockTxn itself. I'm...

So far all reported keys are contract storage keys. An example 2020-05-08T14:34:56.503728535+02:00 WARN Background Snapshott cfxcore::sto - In MptCursorRw, non-existing key [136, 81, 52, 212, 29, 92, 85, 241, 46,...

I'm running a debug build to verify whether there is a bug or not, then close this issue

The behavior seems expected, but I'd like to introduce a change, to prevent deleting a non-existent key when committing an epoch. This could reduce 90% of the complain in the...

Action item: a change, to prevent deleting a non-existent key & prevent committing unchanged data when committing an epoch @yangzhe1990 or @zimpha