-
Notifications
You must be signed in to change notification settings - Fork 465
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
[Feature]: Limit span logs by the select time range (zoom) #2002
Comments
Hey! I would like to contribute on this issue! I am new to this project. Could you please let me know where can I access the logs? Is it the traces on the home you are talking about? @SOF3 |
@khareyash05 the logs appear as vertical ticks in the trace view. A sample trace with logs can be produced via the HotROD demo app. |
Example from https://kubewharf.io/kelemetry/trace-deployment: |
@yurishkuro "Collapse" doesn't sound exactly relevant here. GitHub collapses diffs because git diffs have multiple disjoint intervals of lines, while Jaeger only shows a single continuous interval at a time, so there are no "holes" to collapse. Maybe a toggle to show/hide out-of-range logs would be sufficient (although personally I think it's good enough to just indicate e.g. "n more logs before|after selected range" using the |
Requirement
I often have large spans with a lot (hundreds or thousands) of logs over a wide time range. I want to limit log lists to the dynamically selected time range so that I can quickly view log details without scrolling a lot or searching a smaller trace.
Problem
While I can identify the events I want by hovering over the event ticks on the span, I cannot view the fields of the logs from the hover (my logs often sometimes have large JSON documents). I need to expand the logs below, which means I have to locate the timestamp and binary search the corresponding log entry again.
Proposal
When I zoom in to a smaller time range, the logs beyond the time range should be hidden from the logs list to reduce noise. Currently the table caption is
Logs (N)
. We could change it to:Open questions
No response
The text was updated successfully, but these errors were encountered: