Avoid N+1 queries when sending notifications for reactions (#4822) [deploy]

This commit is contained in:
Molly Struve 2019-11-14 14:36:20 -06:00 committed by GitHub
parent 2ca23c8983
commit 672d64c2ca
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23

View file

@ -24,7 +24,7 @@ module Notifications
reaction_siblings = Reaction.where(reactable_id: reaction.reactable_id, reactable_type: reaction.reactable_type).
where.not(reactions: { user_id: reaction.reactable_user_id }).
includes(:reactable).
includes(:reactable, :user).
order("created_at DESC")
aggregated_reaction_siblings = reaction_siblings.map { |reaction| { category: reaction.category, created_at: reaction.created_at, user: user_data(reaction.user) } }