Task 7 #46
Open
Task 7 #46
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
spajic
approved these changes
Mar 22, 2025
| В нашем проекте тесты (6594) проходят за примерно 5,5 минут на CI (локально около 3 минут). | ||
|
|
||
| Чуть меньше года назад была итерация по ускорению тестов, результат которой: | ||
| 30 минут -> 4 минуты (CI) |
|
|
||
| Чуть меньше года назад была итерация по ускорению тестов, результат которой: | ||
| 30 минут -> 4 минуты (CI) | ||
| 8,5 минут -> 3 минуты (локально) |
|
|
||
| ## Формирование метрики | ||
| Для того, чтобы понимать, дают ли мои изменения положительный эффект на быстродействие программы я придумал использовать такую метрику: | ||
| среднее время выполнения тестов за 3 запуска на локальной машине должно укладываться в 2 минуты |
There was a problem hiding this comment.
сорри за духоту, но это опять не метрика; среднее время за 3 запуска это метрика; а 2 минуты это бюджет;
|
|
||
| name total top-level total time time per call top-level time | ||
| было: company 2075 1268 24.2947s 0.0117s 14.6411s | ||
| стало: company 2075 1268 18.5263s 0.0089s 10.9690s |
There was a problem hiding this comment.
да, можно в этом отчёте вообще посмотреть по time per call, какие фабрики сильно медленнее остальных и обратить на них внимание
| Total: 19893 | ||
| Total top-level: 12747 | ||
| Total time: 01:36.860 (out of 03:06.752) | ||
| Total uniq factories: 110 |
There was a problem hiding this comment.
раньше не было про uniq factories, клёвое нововведение
| user 1629 1604 20.2135s 0.0124s 20.1218s | ||
| Total: 19893 | ||
| Total top-level: 12747 | ||
| Total time: 01:36.860 (out of 03:06.752) |
There was a problem hiding this comment.
ахах, капец, половина от всех тестов юзеры создаются
|
|
||
| user 1035 1010 13.4071s 0.0130s 13.3243s | ||
| Total: 18057 | ||
| Total top-level: 12143 |
There was a problem hiding this comment.
треть в каскадах получается
| Took 67 seconds (1:07) | ||
| Took 63 seconds (1:03) | ||
|
|
||
| вполне себе успех, уложились в бюджет)) |
| вполне себе успех, уложились в бюджет)) | ||
|
|
||
| ## Результаты | ||
| В результате проделанной оптимизации наконец удалось уложиться в бюджет в 120 секунд на локальной машине |
There was a problem hiding this comment.
да ближе даже к одной минуте, отлично!
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.