select('max(id) as id').from(where(user_id: user.id)).group(:issue_id) の from は必要か

Question

この質問は、STI化で起きた障害に関わる質問の、別の解決アプローチに関するものです。
復数のIssueそれぞれに(復数の)Userが復数のMessageをつけることができます。
Message belongs_to issue, belongs_to user
Issue has_many messages,
ですが IssueとUserとは関連つけられて居ません。

ここで、
あるUserがつけたMessageの中でIssuer毎に一番最後のMessage(のid)を集める　のをどうやるか、ということなのですが。
こういう式が作られていました
select('max(id) as id').from(where(user_id: user.id)) .group(:issue_id).map(&:id)
「Messageの中からUserが作ったものだけ抜き出して temp table」を作り、それを対象に Issueでまとめる　という意図と思います。
これと
select('max(id) as id').where(user_id: user.id) .group(:issue_id).map(&:id)
で得られるものに違いが有るでしょうか。違いが有る可能性が有るでしょうか。
group と where の評価順に自信が無くなってしまって、後者でよいと言い切るのが不安。。

各々のSQLは次の様になりました。(type='IssueMessage' はSTIの関係で追加されています。）
前者
SELECT max(id) as id
FROM (SELECT 'issue_messages'.* FROM 'issue_messages' WHERE 'issue_messages'.'type' = 'IssueMessage' AND 'issue_messages'.'user_id' = 1)
subquery WHERE 'issue_messages'.'type' = 'IssueMessage' GROUP BY 'issue_id'

後者
SELECT max(id) as id FROM 'issue_messages' WHERE 'issue_messages'.'type' = 'IssueMessage' AND 'issue_messages'.'user_id' = 1
GROUP BY 'issue_messages'.'issue_id'

Answer 1

前者も後者もやっていることは冗長です。

簡単に言えば、ユーザーid1番の最新の口コミ情報を求めているのだとしたら、いちいち
口コミ情報をユーザーごとにグループ化する必要ないです。

前者
ユーザーid1の最新の口コミ情報の中から、口コミ情報をグループ化して、その中でユーザーid1
の最新情報を取得する

→つまり、最初の検索条件で既に絞り込んでいるのに、そこからグループ化などと面倒な動作に
なってます

後者
口コミ情報の中から、ユーザーごとの口コミ情報にわけてから、ユーザーid1の最新情報を取得する

→検索条件がユーザーid1だけなのでユーザーごとにグループ化する必要がないように思います。

なので、
select('max(id) as id').where(user_id: user.id)
だけでいいと思います

Comments

[winterboum]

いや、GROUP BY 'issue_messages'.'issue_id'　です。userでgroupではないです

前者はやはり冗長ですよね。

[FKM]

再確認しました。口コミ情報のグループということですね。
前者は明らかに冗長で、非効率ですね