Skip to content
Permalink
Browse files

Improve account media query (#10121)

  • Loading branch information...
abcang authored and Gargron committed Feb 26, 2019
1 parent 04197a2 commit aa63c3e22af013413dcab1e52cd5e66a0d9723e3
Showing with 3 additions and 3 deletions.
  1. +3 −3 app/controllers/api/v1/accounts/statuses_controller.rb
@@ -50,9 +50,9 @@ def account_media_status_ids
# Also, Avoid getting slow by not narrowing down by `statuses.account_id`.
# When narrowing down by `statuses.account_id`, `index_statuses_20180106` will be used
# and the table will be joined by `Merge Semi Join`, so the query will be slow.
Status.joins(:media_attachments).merge(@account.media_attachments).permitted_for(@account, current_account)
.paginate_by_max_id(limit_param(DEFAULT_STATUSES_LIMIT), params[:max_id], params[:since_id])
.reorder(id: :desc).distinct(:id).pluck(:id)
@account.statuses.joins(:media_attachments).merge(@account.media_attachments).permitted_for(@account, current_account)
.paginate_by_max_id(limit_param(DEFAULT_STATUSES_LIMIT), params[:max_id], params[:since_id])
.reorder(id: :desc).distinct(:id).pluck(:id)
end

def pinned_scope

0 comments on commit aa63c3e

Please sign in to comment.
You can’t perform that action at this time.