tikv / yatp

Yet another thread pool in rust for both callbacks or futures.
Apache License 2.0
135 stars 32 forks source link

fix task total execute time tracker #71

Closed glorv closed 1 year ago

glorv commented 1 year ago

Fix the bug that the task total execution time is not updated on time. The bug is introduced by #67. This bug can affect the performance impact of big task for small tasks. I test the performace impact by running sysbench oltp_read_only(--thread=50 --range-selects) as small tasks and run go-tpc tpch --sf=10 as big tasks. Before:

before

After

after

The throughput impact is much better and the tail latency is slightly better.

glorv commented 1 year ago

@BusyJay @sticnarf PTAL, thanks