Commit 342e902
committed
fix: measure individual call durations in sequential phase of concurrency decorator
The old approach timed the entire sequential loop (including event loop
scheduling overhead between iterations), while the concurrent phase
used asyncio.gather which has a fast-path with minimal inter-task
overhead. This asymmetry inflated the sequential/concurrent ratio,
causing false positives in concurrency gain detection.
Sum per-iteration durations instead, excluding event loop scheduling
overhead from the measurement.1 parent dbe5e1a commit 342e902
3 files changed
Lines changed: 13 additions & 6 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
181 | 181 | | |
182 | 182 | | |
183 | 183 | | |
184 | | - | |
| 184 | + | |
| 185 | + | |
| 186 | + | |
185 | 187 | | |
186 | 188 | | |
187 | | - | |
188 | 189 | | |
| 190 | + | |
189 | 191 | | |
190 | | - | |
| 192 | + | |
191 | 193 | | |
192 | 194 | | |
193 | 195 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
1643 | 1643 | | |
1644 | 1644 | | |
1645 | 1645 | | |
| 1646 | + | |
1646 | 1647 | | |
1647 | 1648 | | |
1648 | | - | |
1649 | 1649 | | |
| 1650 | + | |
1650 | 1651 | | |
1651 | | - | |
| 1652 | + | |
1652 | 1653 | | |
1653 | 1654 | | |
1654 | 1655 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
221 | 221 | | |
222 | 222 | | |
223 | 223 | | |
224 | | - | |
| 224 | + | |
225 | 225 | | |
226 | 226 | | |
| 227 | + | |
| 228 | + | |
| 229 | + | |
| 230 | + | |
227 | 231 | | |
228 | 232 | | |
229 | 233 | | |
| |||
0 commit comments