jaeger-ui icon indicating copy to clipboard operation
jaeger-ui copied to clipboard

[Feature]: Limit span logs by the select time range (zoom)

Open SOF3 opened this issue 1 year ago • 4 comments
trafficstars

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:

Logs (m of N) (show more) (show all)

Open questions

No response

SOF3 avatar Nov 24 '23 05:11 SOF3

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 Thank you

khareyash05 avatar Nov 26 '23 05:11 khareyash05

@khareyash05 the logs appear as vertical ticks in the trace view. A sample trace with logs can be produced via the HotROD demo app.

yurishkuro avatar Nov 26 '23 15:11 yurishkuro

Example from https://kubewharf.io/kelemetry/trace-deployment: image I only selected 2.23s..4.5s, so the last three logs should not be displayed

SOF3 avatar Nov 27 '23 02:11 SOF3

@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 AccordianLogs--foter class).

SOF3 avatar Nov 27 '23 02:11 SOF3