mstange / samply

Command-line sampling profiler for macOS and Linux
Apache License 2.0
1.96k stars 48 forks source link

Add --time-range option for import for Windows #261

Closed vvuk closed 1 week ago

vvuk commented 2 weeks ago

Add a --time-range option for import on Windows to allow specifying a time range to include in the profile.json from the ETL file. Processing still tracks process/thread/image lifecycle, but samples and events are only added if they're within this time range. This is very helpful in exploring a very large trace to avoid running into profile.json file size issues. (Could be implemented on other platforms as well, though typically the profiles there are much smaller.)

vvuk commented 2 weeks ago

Hm how is clippy blowing up on linux for some of this stuff now

mstange commented 2 weeks ago

New Rust version came out and added new warnings. I'll fix it and you can rebase.

mstange commented 1 week ago

Merging so that we don't have to keep rebasing it. We can make any tweaks as follow-ups.