Skip to content
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

Show indicators for markers in the thread timeline #50

Closed
mstange opened this issue Nov 25, 2016 · 5 comments
Closed

Show indicators for markers in the thread timeline #50

mstange opened this issue Nov 25, 2016 · 5 comments

Comments

@mstange
Copy link
Contributor

mstange commented Nov 25, 2016

No description provided.

@gregtatum
Copy link
Member

I'm not sure what this issue is getting at. Isn't this already implemented?

@mstange
Copy link
Contributor Author

mstange commented Mar 8, 2017

The thing that we already have shows "tracing" markers (i.e. markers with a start time and an end time), but not point-of-time markers. For point-of-time markers, the old cleopatra would overlay vertical lines onto the ThreadStackGraph, and every line had a piece of text with the marker name attached to it.

I've written the start of a component that implements this (ThreadMarkerOverlay), but it's not being used, and I'm not sold on the idea of squeezing even more information into the stuff at the top.
We do need to display it somewhere, though. Mike Conley is the person who asks me about this the most.

Examples for such markers are the "Invalidate" and "Bailout" markers from JS (too many to be useful), "AsyncTabSwitch:Start", "AsyncTabSwitch:Finish", "AsyncTabSwitch:SpinnerShown", "AsyncTabSwitch:SpinnerHidden", and some others that you can find with these code searches.

@gregtatum
Copy link
Member

Seems like a good candidate for moving to the Timeline.

@gregtatum
Copy link
Member

Closing this for now, it's old and I'm not sure what we want to do here.

@julienw
Copy link
Contributor

julienw commented Nov 29, 2017

I believe we can already achieve this by clicking on a marker as it will add the selection's vertical line. (we need to fix the alignment bug of course)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants