apache / skywalking-ui

[ARCHIVED] The web UI for skywalking APM
https://skywalking.apache.org/
Apache License 2.0
273 stars 161 forks source link

Make the new trace view better #213

Closed wu-sheng closed 5 years ago

wu-sheng commented 5 years ago

I have two suggestions to make the new trace view better.

  1. Add a new filter, highlight the top N duration span. This duration should exclude all child spans's duration. Calculate by UI. And notice, sometime, this may trigger negative duration,just ignore it.
  2. Add a new filter, highlight the top N parent spans which have the max children spans.
wu-sheng commented 5 years ago

Ref #215

wu-sheng commented 5 years ago

@TinyAllen i will keep this open to track another feature requirement. Support broken trace segments in new trace view. OK? But please send the feature sync separated in two pull requests for main repo.

wu-sheng commented 5 years ago

image

This view height should be self-adaption. @TinyAllen Could you consider optimize this?

wu-sheng commented 5 years ago

Fixed in #217