[PATCH 3 of 3] run-tests: add --time option to log times for each test
Matt Mackall
mpm at selenic.com
Wed Nov 14 15:05:04 CST 2012
On Mon, 2012-11-12 at 13:08 -0800, Siddharth Agarwal wrote:
> On 11/12/2012 08:00 AM, Matt Mackall wrote:
> > Wall time by itself isn't terribly useful, especially when using -j. For
> > instance, on my 2 core, 4 thread laptop i7, I find that -j 8 runs
> > through the test suite fastest.
> >
> > Much more interesting is user+sys time from os.times, though it's often
> > useful to have these split out.
>
> I did consider that while writing the patch, but decided not to do so
> because:
> (1) User+system time doesn't capture time spent waiting on I/O, which
> seemed to me to be important.
And yet, without user+sys, wall time is not very meaningful.
> (2) We'd need to figure out user+sys time of the process's children. The
> documentation for os.times() [1] indicates that this can't be done portably.
And yet, without user+sys, wall time is not very meaningful.
Please consider adding user and sys or user+sys to make this feature
useful.
--
Mathematics is the supreme nostalgia of our time.
More information about the Mercurial-devel
mailing list