Issue 13839: -m pstats should combine all the profiles given as arguments (original) (raw)

Issue13839

Created on 2012-01-23 03:38 by anacrolix, last changed 2022-04-11 14:57 by admin. This issue is now closed.

Files
File name Uploaded Description Edit
m-pstats-merge-profiles.patch anacrolix,2012-01-23 03:38 review
Messages (4)
msg151795 - (view) Author: Matt Joiner (anacrolix) Date: 2012-01-23 03:38
Frequently when profiling multiple threads, I need to combine several dump stat files. Currently -m pstats reads the profiling data at only the first path given. It should merge all the profiling data from all the paths given. $ python3.3 -m pstats prof/5506-7f00f* Minimalist patch attached.
msg152682 - (view) Author: Éric Araujo (eric.araujo) * (Python committer) Date: 2012-02-05 12:34
This sounds like a reasonable request. The patch needs tests, probably using test.script_helper.assert_python_ok.
msg155766 - (view) Author: Roundup Robot (python-dev) (Python triager) Date: 2012-03-14 16:51
New changeset 3a5a0e7d38c5 by Antoine Pitrou in branch 'default': Issue #13839: When invoked on the command-line, the pstats module now accepts several filenames of profile stat files and merges them all. http://hg.python.org/cpython/rev/3a5a0e7d38c5
msg155767 - (view) Author: Antoine Pitrou (pitrou) * (Python committer) Date: 2012-03-14 16:54
I've committed the patch. Eric, adding a test suite for pstats' command-line interface is a separate concern.
History
Date User Action Args
2022-04-11 14:57:25 admin set github: 58047
2012-03-14 16:54:39 pitrou set status: open -> closednosy: + pitroumessages: + resolution: fixedstage: test needed -> resolved
2012-03-14 16:51:38 python-dev set nosy: + python-devmessages: +
2012-02-05 12:34:15 eric.araujo set nosy: + eric.araujomessages: +
2012-01-27 23:06:08 terry.reedy set nosy: + georg.brandlstage: test neededversions: - Python 2.6, Python 3.1, Python 2.7, Python 3.2, Python 3.4
2012-01-23 03:38:16 anacrolix create