Closed
Description
openedon Jan 6, 2020
Problem - what in Jaeger blocks you from solving the requirement?
jaeger/storage/spanstore/interface.go
Lines 27 to 29 in 009b7f5
The spanstore.Writer
interface does not propagate context.
Proposal - what do you suggest to solve the problem or improve the existing situation?
Add a new WriteSpanWithContext(ctx context.Context, span *model.Span) error
to propagate context.
Alternatively, we may update to WriteSpan(ctx context.Context, *model.Span) error
.
Unfortunately, both of these are breaking changes. Alternatively, we may create a WriterV2
interface, and supply a converter from Writer
to WriterV2
. Such a converter would simply use ctx.Background()
for the context parameter.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment