dusk-blockchain icon indicating copy to clipboard operation
dusk-blockchain copied to clipboard

Review mempool design as well as testing strategy, and remove any possible bottlenecks

Open jules opened this issue 4 years ago • 1 comments

As the mempool is currently heavily relying on channel-based processing, it stands to reason that we can most likely increase the efficiency by a large margin.

Additionally, the mempool testing strategy should be reviewed, and possibly altered if deemed necessary. In my experience, the current test file may be a bit hard to read when you are new to it, and the singleton-based strategy (with the context object that resets a single instance of the mempool throughout the tests) has caused some regressions in the past that were purely testing-related, even though the mempool itself worked just fine. A review and a possible re-design might save us time in the future of having to catch these harmless regressions and fix them, just to have our pipeline succeed.

jules avatar Mar 26 '20 11:03 jules

This issue depends on phoenix integration to be done first

autholykos avatar Mar 30 '20 08:03 autholykos