Commit 6ba029c
committed
fix: add exclude_list to @trace_class on high-frequency event classes
Addresses #1034 — the @trace_class decorator on EventQueue and related classes
generates 1500+ spans per LLM streaming session from high-frequency internal
methods (enqueue_event, dequeue_event, task_done, etc.).
Added exclude_list to @trace_class on:
- EventQueueLegacy: excludes enqueue_event, dequeue_event, task_done, is_closed
- EventQueueSource: excludes enqueue_event, dequeue_event, task_done, is_closed
- EventConsumer: excludes consume_all
- InMemoryQueueManager: excludes add, get, create_or_tap
This reduces tracing spans from 1500+ to ~53 per session (97% reduction) while
preserving useful RequestHandler-level traces. The existing exclude_list
mechanism in trace_class is used — no new API surface or breaking changes.1 parent c0c6c08 commit 6ba029c
4 files changed
Lines changed: 16 additions & 4 deletions
File tree
- src/a2a/server/events
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
18 | 18 | | |
19 | 19 | | |
20 | 20 | | |
21 | | - | |
| 21 | + | |
| 22 | + | |
| 23 | + | |
| 24 | + | |
22 | 25 | | |
23 | 26 | | |
24 | 27 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
93 | 93 | | |
94 | 94 | | |
95 | 95 | | |
96 | | - | |
| 96 | + | |
| 97 | + | |
| 98 | + | |
| 99 | + | |
97 | 100 | | |
98 | 101 | | |
99 | 102 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
20 | 20 | | |
21 | 21 | | |
22 | 22 | | |
23 | | - | |
| 23 | + | |
| 24 | + | |
| 25 | + | |
| 26 | + | |
24 | 27 | | |
25 | 28 | | |
26 | 29 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
9 | 9 | | |
10 | 10 | | |
11 | 11 | | |
12 | | - | |
| 12 | + | |
| 13 | + | |
| 14 | + | |
| 15 | + | |
13 | 16 | | |
14 | 17 | | |
15 | 18 | | |
| |||
0 commit comments