Optimization #121
Open
Optimization #121
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 7, 2025
| @@ -30,7 +30,7 @@ | |||
|
|
|||
| Нужно оптимизировать механизм перезагрузки расписания из файла так, чтобы он импортировал файл `large.json` **в пределах минуты**. | |||
|
|
|||
| `rake reload_json[fixtures/large.json]` | |||
| `rake "reload_json[fixtures/large.json]"` | |||
| @@ -2,6 +2,6 @@ class TripsController < ApplicationController | |||
| def index | |||
| @from = City.find_by_name!(params[:from]) | |||
| @to = City.find_by_name!(params[:to]) | |||
| @trips = Trip.where(from: @from, to: @to).order(:start_time) | |||
| @trips = Trip.preload(:from, :to, bus: :services).where(from: @from, to: @to) | |||
There was a problem hiding this comment.
а мы тут сортировку не потеряли?
| cities: Set.new, | ||
| buses: [], | ||
| services: Set.new | ||
| }.tap do |data| |
There was a problem hiding this comment.
кстати вот в контексте профилирования и оптимизации не очень удобно работать с tap
в отчётах профилировщиков это может выглядеть как - вся работа была в блоке tap (а что тормозило внутри этого блока, и какой именно это был tap в случае если их несколько - непонятно)
| <%= render "delimiter" %> | ||
| <% end %> | ||
| <div class="trips-container"> | ||
| <%= render partial: "trip", collection: @trips, spacer_template: "delimiter" %> |
| Количество SQL запросов | ||
| Время выполнения SQL запросов | ||
|
|
||
| Для контроля корректности работы скрипта после внесения изменений, я сначала создал тест (spec/services/reloader_spec.rb) |
| ## Гарантия корректности | ||
| Для обеспечения корректности оптимизации: | ||
|
|
||
| Написан интеграционный тест, проверяющий идентичность отображения данных из example.json до и после оптимизации |
|
|
||
| ## Итого: | ||
| Время выполнения для small.json уменьшилось с 27 секунд до 0.3 секунд. | ||
| Время выполнения для large.json составила 12 секунд, что укладывается в бюджет. |
| Trip Count (21.2ms) SELECT COUNT(*) FROM "trips" WHERE "trips"."from_id" = $1 AND "trips"."to_id" = $2 [["from_id", 252], ["to_id", 254]] | ||
| ``` | ||
| - Логи rails и rack-mini-profiler | ||
| - Проверил как выполняется запрос в pg_hero, используется Seq Scan, Pg_hero предложил добавить индекс CREATE INDEX CONCURRENTLY ON trips (from_id, to_id) |
There was a problem hiding this comment.
👍 на мой взгляд это просто супер-удобно, что он прям предлагает сам конкретно что надо делать, да ещё такие не особо тривиальные вещи как составной индекс
| ``` | ||
| - Логи rails и rack-mini-profiler | ||
| - Проверил как выполняется запрос в pg_hero, используется Seq Scan, Pg_hero предложил добавить индекс CREATE INDEX CONCURRENTLY ON trips (from_id, to_id) | ||
| - Скорость выполнения запроса сократилась с 21ms до 1.5ms |
There was a problem hiding this comment.
Also, там count-запрос не нужен если мы уж всё равно грузим все объекты для показа на странице (так как тут нет никакой пагинации)
| - В логах rack-mini-profiler видно множественный рендеринг одного и того же partial _services.html.erb | ||
| - Текущая реализация в index.html.erb использует цикл с отдельным рендерингом для каждой поездки | ||
| - Заменили на использование collection rendering | ||
| - Время рендеринга страницы сократилось с 2с до 1с |
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.