Плохо масштабируемые рельсы Запрос ActiveRecord

Question

У меня есть следующий запрос ActiveRecord. Он работал нормально, когда количество сообщений было 50 в моей тестовой среде, но когда мы ввели его в производство, а количество сообщений выросло до 5000, время ответа приблизилось к 30 секундам. Нехорошо.

Как я могу сделать этот запрос более эффективно, так что держите его быстрым, так как количество сообщений растет. Запрос находит все предупреждения из всех сообщений, если они еще не завершены.

class AlertsController < ApplicationController 
    before_filter :get_user 
    respond_to :json, :html 

def index 
    @messages = current_user.messages.where(:active => true).order("created_at ASC") 

    @alerts = Array.new 
    @messages.each do |message| 
     if (message.alerts.count > 0) 
      @alerts = @alerts + message.alerts.where(:completed => false) 
     end 
    end 

    respond_to do |format| 
     format.html 
     format.json 
    end 
    end 

end 


class Alert < ActiveRecord::Base 
    belongs_to :message 

class Message < ActiveRecord::Base 
    has_many :alerts, dependent: :destroy 

Answer 1

здание от ответа Джесси, немного лучше было бы создать область только выбрать pending_alerts , таким образом вы пропускаете меньше данных и позволяете базе данных выполнять большую часть работы.

class Message < ActiveRecord::Base 
    scope :pending_alerts, lambda { 
    alerts.where(:completed => false) 
    } 
} 

Тогда в контроллере:

@messages = current_user.messages.includes(:pending_alerts).where(:active => true).order("created_at ASC") 
@alerts = @messages.map { |m| m.alerts } 

Answer 2

Это проблема с n + 1. Документация здесь (поиск Solution to N + 1 queries problem)

@messages = current_user.messages.includes(:alerts).where(:active => true).order("created_at ASC") 

Затем, чтобы сделать @alerts проще:

@alerts = @messages.map do |message| 
    message.alerts.select {|alert| !alert.completed} 
end