chore: 4th step case b

Author: potashin

app/controllers/trips_controller.rb

@@ -2,7 +2,6 @@ class TripsController < ApplicationController
   def index
     @from = City.find_by_name!(params[:from])
     @to = City.find_by_name!(params[:to])
-    @services = Service.pluck(:id, :name).to_h
-    @trips = Trip.where(from: @from, to: @to).preload(bus: :bus_services).order(:start_time)
+    @trips = Trip.where(from: @from, to: @to).preload(bus: :services).order(:start_time)
   end
 end

app/models/bus.rb

@@ -21,8 +21,7 @@ class Bus < ApplicationRecord
   ].freeze
 
   has_many :trips
-  has_many :bus_services, class_name: 'BusService'
-  has_many :services, through: :bus_services
+  has_and_belongs_to_many :services, join_table: :buses_services
 
   validates :number, presence: true, uniqueness: true
   validates :model, inclusion: { in: MODELS }

app/models/bus_service.rb

@@ -19,8 +19,7 @@ class Service < ApplicationRecord
     'Можно не печатать билет',
   ].freeze
 
-  has_many :service_buses
-  has_many :buses, through: :service_buses
+  has_and_belongs_to_many :buses, join_table: :buses_services
 
   validates :name, presence: true
   validates :name, inclusion: { in: SERVICES }

app/models/service.rb

@@ -1 +1 @@
-<li><%= "#{@services[bus_service.service_id]}" %></li>
+<li><%= "#{service.name}" %></li>

app/views/trips/_service.html.erb

@@ -4,8 +4,11 @@
     <li><%= "В пути: #{trip.duration_minutes / 60}ч. #{trip.duration_minutes % 60}мин." %></li>
     <li><%= "Цена: #{trip.price_cents / 100}р. #{trip.price_cents % 100}коп." %></li>
     <li><%= "Автобус: #{trip.bus.model} №#{trip.bus.number}" %></li>
-    <% if trip.bus.bus_services.present? %>
-        <%= render "services", bus_services: trip.bus.bus_services %>
+    <% if trip.bus.services.present? %>
+        <li>Сервисы в автобусе:</li>
+        <ul>
+        <%= render partial: "service", collection: trip.bus.services %>
+        </ul>
     <% end %>
 </ul>
 

app/views/trips/_services.html.erb

@@ -23,46 +23,38 @@
 Вот как я построил `feedback_loop`: профилирование - изменение кода - тестирование – бенчмаркинг – откат при отсутствии разницы от оптимизации/сохранение результатов
 
 ## Вникаем в детали системы, чтобы найти главные точки роста
-Для того, чтобы найти "точки роста" для оптимизации я воспользовался rack mini profiler
+Для того, чтобы найти "точки роста" для оптимизации я воспользовался rack mini profiler, bullet
 
 Вот какие проблемы удалось найти и решить
 
 ### Ваша находка №1
-- rack mini profiler показал, что N+1 проблему `SELECT  "buses".* FROM "buses" WHERE "buses"."id" = $1 LIMIT $2; ` + `SELECT "services".* FROM "services" INNER JOIN "buses_services" ON "services"."id" = "buses_services"."service_id" WHERE "buses_services"."bus_id" = $1;`  в `trips/_services.html.erb` (на 1004 рейса 690 sql buses + services)
+- bullet показал, что N+1 проблему `SELECT  "buses".* FROM "buses" WHERE "buses"."id" = $1 LIMIT $2; ` + `SELECT "services".* FROM "services" INNER JOIN "buses_services" ON "services"."id" = "buses_services"."service_id" WHERE "buses_services"."bus_id" = $1;`  
 - делаю `.preload(bus: :services)` для `trips`
 - метрика снизилась до 6 секунд
 - количество sql запросов для `trips/index.html.erb` сократилось до 12
 
-### Ваша находка 2
+### Ваша находка № 2
 - rack mini profiler (и логи веб сервера) показал, что основное время тратится на рендеринг шаблонов `trips/index.html.erb	2091.1
 4917.9`.
 - рендереринг всех коллекций через `render partial:`
-- метрика снизилась до 1 секунды
+- метрика снизилась до 600мс
 - `Rendering: trips/index.html.erb	716.2 975.7`
 
-### Ваша находка 3
-- rack mini profiler показал, что основное время занимает запрос в сервисы (около 400мс).
-- вместо habtm делаю промежуточную таблицу, для нее и делаю eager loading вмсто services. в контроллере достаю весь и собираю хэш-справочник (id=>name) сервисов и использую его во вью.
-- метрика особенно не снизилась
-- запрос пропал из логов профилировщика (вместо него показывается запрос в сервисы, около 25мс, что 8 раз меньше)
-
-### Ваша находка 4
+### Ваша находка № 3
 - rack mini profiler показал, что производится и запрос по count, и идентичный по выборке данных.
 - меняю count на length.
 - метрика особенно не снизилась
 - запрос count пропал из логов профилировщика
 
-### Ваша находка № 5
-- rack mini profiler показал, что долго выполняются запросы `SELECT "buses_services".* FROM "buses_services" WHERE "buses_services"."bus_id" IN ($1, <...>` (делаю explain, он показывает `Seq Scan on buses_services` как основную точку роста),``
-- Добавляю индекс на `bus_id`
-- как изменилась метрика
-- как изменился отчёт профилировщика - исправленная проблема перестала быть главной точкой роста?
-
-
+### Ваша находка № 4
+- rack mini profiler показал, что долго выполняются запрос `  SELECT "trips".* FROM "trips" WHERE "trips"."from_id" = $1 AND "trips"."to_id" = $2 ORDER BY "trips"."start_time" ASC` (делаю explain, он показывает `Seq Scan on trips` как основную точку роста),
+- Добавляю индекс на связку `from_id`/`to_id`/`start_time`
+- метрика особенно не снизилась
+- время работы запроса снизилось с 18 до 2.5мс
 
 ## Результаты
 В результате проделанной оптимизации наконец удалось обработать файл с данными.
-Удалось улучшить метрику системы с *того, что у вас было в начале, до того, что получилось в конце* и уложиться в заданный бюджет.
+Удалось улучшить метрику системы с 13.3с до 0.6с.
 
 ## Защита от регрессии производительности
 Для защиты от потери достигнутого прогресса при дальнейших изменениях программы *о performance-тестах, которые вы написали*

app/views/trips/_trip.html.erb

@@ -0,0 +1,7 @@
+class AddFromIdToIdIndexToTrips < ActiveRecord::Migration[5.2]
+  disable_ddl_transaction!
+
+  def change
+    add_index(:trips, %i[from_id to_id start_time], order: {start_time: :asc}, algorithm: :concurrently)
+  end
+end

case-study-b.md

@@ -10,7 +10,7 @@
 #
 # It's strongly recommended that you check this file into your version control system.
 
-ActiveRecord::Schema.define(version: 2019_03_30_193044) do
+ActiveRecord::Schema.define(version: 2024_05_15_205445) do
 
   # These are extensions that must be enabled in order to support this database
   enable_extension "plpgsql"
@@ -40,6 +40,7 @@
     t.integer "duration_minutes"
     t.integer "price_cents"
     t.integer "bus_id"
+    t.index ["from_id", "to_id", "start_time"], name: "index_trips_on_from_id_and_to_id_and_start_time"
   end
 
 end