-
Notifications
You must be signed in to change notification settings - Fork 253
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Missing dropped_spans_count #589
Comments
Where exactly (in which proto message) would this be recorded and who would know the dropped span count? There is no |
As I understand it, it could be either ScopeSpans or ResourceSpans. Basically, it's the burden of a Tracer that has a name that uniquely identifies the instrumentation scope. So I would say that ScopeSpans is more appropriate than ResourceSpans.
Tracer, because it's the only API for creating a Span.
Sorry, didn't understand this question. But Tracer can incriminate this counter every time it fails to create a new Span. |
@leonidmanieiev maybe open-telemetry/oteps#249 is what you're looking for? |
Will this pipeline monitoring proposal eventually become v2/trace.proto or where will it go? |
@leonidmanieiev I think I understand what you are looking for, but this needs more details. This is not just a proto change, it will require changes in the SDKs as well. I think an OTEP would be warranted in this case. I do not know whether the OTEP will be accepted, but that's where I would start. |
@tigrannajaryan Thank you, I'll try to propose in OTEP. Closing this issue. |
@leonidmanieiev I recommend attending Specification SIG meeting to gauge the community interest before you start working on the OTEP. I would also open an issue in the spec repo. It will get triaged and if it gets accepted only after that it makes sense to go ahead. If the issue is not accepted there is no point in proposing an OTEP. |
v1 trace.proto describes the following counters:
dropped_events_count
+dropped_attributes_count
of the eventdropped_links_count
+dropped_attributes_count
of the linkdropped_attributes_count
of the spandropped_spans_count
is missing, although it'd be as useful as the others because, as with attributes, spans can be discarded due to long names or because there are too many of them.dropped_spans_count
?The text was updated successfully, but these errors were encountered: