Add case study #134
Add case study #134
Conversation
|
|
||
| Вот как я построил `feedback_loop`: | ||
| - вношу изменение | ||
| - выполняю тест который проверяет, стала ли программа в 2 раза быстрее |
There was a problem hiding this comment.
в 2 раза быстрее конечно не всегда получается
|
|
||
| - как вы решили её оптимизировать | ||
|
|
||
| Во-первых, мы будем выбирать сессии пользователя не по каждому пользователю снова и снова - а пробежимся с этой целью по сессиям только один раз, при этом раскидывая сессии по пользователям. |
There was a problem hiding this comment.
тем самым делается самая главная оптимизация, а именно сложность снижается с O(N^2) до O(N)
|
|
||
| - как изменилась метрика | ||
|
|
||
| 2_800ms -> 480ms |
There was a problem hiding this comment.
тут хорошо бы увеличить объём данных, чтобы программа успевала поработать секунд 5
| - какой отчёт показал главную точку роста | ||
|
|
||
| Смотрим flat + graph. Array#each остается одним из лидеров. | ||
| Прежде чем приступать к оптимизации, попробуем выделить фрагменты программы в отдельные методы, чтобы было удобнее профилировать. |
There was a problem hiding this comment.
да, просто each мало о чём говорит, легко свалиться в угадывание, лучше потратить время чтобы уточнить откуда именно идёт большинство вызовов
|
|
||
| - как вы решили её оптимизировать | ||
|
|
||
| Осуществлять поиск кол-ва никальных браузеров с помощью uniq |
There was a problem hiding this comment.
уникальные браузеры тут удобно и эффективно через Set собирать
| ``` | ||
| - как вы решили её оптимизировать | ||
|
|
||
| При формировании 'dates' использование `Date.parse` (дорогого) с последующим переводом в `iso8601` кажется избыточным убрали (имеющийся тест проходит). |
|
|
||
| - как изменился отчёт профилировщика - исправленная проблема перестала быть главной точкой роста? | ||
|
|
||
| `merge` ушел из отчета. Ура! теперь мы попадаем в бюджет. |
| expect { work(file_path) }.to perform_under(ms).ms.warmup(warmup_seconds).times.sample(10).times | ||
| end | ||
|
|
||
| it 'is trying to be better ' do |
|
|
||
| report['uniqueBrowsersCount'] = uniqueBrowsers.count | ||
| report['totalSessions'] = users.map(&:sessions_count).sum |
There was a problem hiding this comment.
map тут создаст лишний массив, лучше бы без этого обойтись
|
|
||
| File.write('result.json', "#{report.to_json}\n") | ||
| ensure | ||
| GC.enable |
There was a problem hiding this comment.
Особо смысла нет, так как программа уже закрывается всё равно
No description provided.