kornilova203 / FlameViewer

Tool for flamegraphs visualization
MIT License
80 stars 7 forks source link

Moving to Compressed Flamegraphs for uploaded JFR files #50

Closed brtubb-patagonia closed 5 years ago

brtubb-patagonia commented 5 years ago

Followup from the failed PR here: https://github.com/kornilova-l/FlameViewer/pull/49#pullrequestreview-180332813

Squashed commits from my dev branch into a new branch to remove confusing commits that did opposite things (like restoring files). Going forward, I'll make fixes in my dev branch and squash before merging back into this PR. Sorry if I'm making things messy - I'm ok with Git, but don't often do PRs so may be bad with etiquette. Happy to adjust as needed!

kornilova203 commented 5 years ago

I'll get back to it today

kornilova203 commented 5 years ago

I made converters API more consistent #51, I'll merge it into master and rebase jfr-to-cflamegraph