rails5で検索クエリを最後に一回だけ発行するようにしたい

Question

Rails5にバージョンアップするにあたって、どうやら検索クエリの実行されるタイミングが変わったらしく、検索用のクエリを発行するまでに二回もSQLが吐かれるようになってしまいました。

# app/models/search/user.rb

def matches
  users = ::User.all
  # ほんとはもっと複雑な検索ロジックなのですが、問題の部分だけをピックアップしておきました。
  # とりあえずここの部分が入っているだけで、結構な量のSQLになるので…
 # 「検索ロジック自体に問題がある」というよりかは、「早い段階でSQLを発行されてしまう」こと自体に問題があるように思います
  users = users.preload(corporation: :region)
  users = users.left_joins(:corporation)
  users = users.corporation_name_like(name) if name.present?
  users
end

# app/controllers/user_controller.rb

def search
  @users = Search::User.new(search_params).matches
  # Rails5では、この時点でSQLを一度発行するようになってしまった
  @users = @users.order(:name)
                 .page(params[:page])
                 .per(20)
 # Rails4ではここまで来てようやくSQLを発行していた
end

以前はsearchメソッドの最後に行くまでSQLが発行されなかったので、kaminariのpagenateによるLIMITがかかり、検索時間も全くかからなかったのですが、Rails5にあげてからはmatches関数の後で一回SQLが発行されるようになってしまったがために、すさまじい量のデータを毎回ロードする羽目になってしまいます。

質問というのは、

・どうやったらSQL発行のタイミングを制御できるのか

ということです

お願いいたします

Comments

[moke]

# 検索ロジックを晒していただかないと、Rails5に精通している人しか答えられませんよ。回答がつく機会を逃していると思います。

[dialbird]

mokeさん。ご忠告ありがとうございます。追記します！

Answer 1

恥ずかしながらあんなこと書いておいて検索ロジックの問題でした…
省略した検索ロジックの中に、

return [] if users.blank?

という一行が入っていて、この空判定をするために一度SQLを発行していたのでした。

Rails5になったらSQLが二回も発行されるというのは完全に私の勘違いでした。
誠に失礼いたしました。