action #158511
opencoordination #154777: [saga][epic] Shareable os-autoinst and test distribution plugins
coordination #108527: [epic] os-autoinst wheels for scalable code reuse of helper functions and segmented test distributions
Try to optimize the startup time of isotovideo more
0%
Description
Motivation¶
Based on #157339 We had cases like that in the past and should try to identify the bottleneck of that test t/14-isotovideo again and improve the runtime of in particular isotovideo itself.
Acceptance criteria¶
- AC1: isotovideo consistently starts up quicker
Suggestions¶
- Run
perl -d:NYTProf ./isotovideo --help
andperl -d:NYTProf ./isotovideo .
- Consider trying to improve the runtime of isotovideo itself, not only making the test faster
Updated by okurz 7 months ago
- Copied from action #157339: os-autoinst t/14-isotovideo.t is again taking too long (>20s on my setup) size:M added
Updated by okurz 7 months ago
- Assignee set to okurz
I found that already time perl -e 'use Mojo::Base'
takes a significant 200ms for me. Maybe https://github.com/mojolicious/mojo/commit/7e9a2ad15f3f62686deeca145b6740ab846717b5 shows some potential for improvement. Mojo::Base loads (complete) Mojo::Util with require Mojo::Util
. Not needing Mojo::Util at all would reduce time perl -e 'use Mojo::Base'
for me to 50ms.
Updated by okurz 6 months ago
https://github.com/mojolicious/mojo/pull/2173 merged (finally). It will likely now also take some time until we have a new release version available including that to see if it has a reasonable effect on how we actually use the component in our products.