You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
## Which problem is this PR solving?
Resolves#4375
## Short description of the changes
Expose drop span hook as an option when creating SpanProcessor in
collector so that users have the flexibility to get more insights on
dropped spans in addition to the dropped count.
Signed-off-by: Chen Xu <[email protected]>
Co-authored-by: Yuri Shkuro <[email protected]>
Requirement
As a Jaeger operator, I'd like to have customized handling when spans are dropped due to incoming queue being full on Jaeger Collector.
Problem
The
droppedItemHandler
is configured insidenewSpanProcessor
.jaeger/cmd/collector/app/span_processor.go
Lines 90 to 100 in fa94007
Proposal
Use the existing
Options
struct to add a newOnDroppedSpan
, use a type assertion internally to convert theinterface{}
to a*model.Span
.Open questions
No response
The text was updated successfully, but these errors were encountered: