fade1a4cde
This was not a clear cut n+1 query, given that if you're directly subscribed to all issues that the API is returning you never really need to check for the notes. However, if you're subscribed to _all_ of them, then for each issuable you need to go once to `notes`, and once to `users` (for the authors). By preemtively loading notes and authors, at worst you have 1 extra query, and at best you saved 2n extra queries. We also took advantage of this preloading of notes when counting user notes. |
||
---|---|---|
.. | ||
ci | ||
concerns | ||
hooks | ||
members | ||
project_services | ||
ability_spec.rb | ||
abuse_report_spec.rb | ||
appearance_spec.rb | ||
application_setting_spec.rb | ||
award_emoji_spec.rb | ||
blob_spec.rb | ||
broadcast_message_spec.rb | ||
build_spec.rb | ||
commit_range_spec.rb | ||
commit_spec.rb | ||
commit_status_spec.rb | ||
deploy_key_spec.rb | ||
deploy_keys_project_spec.rb | ||
email_spec.rb | ||
event_spec.rb | ||
external_issue_spec.rb | ||
forked_project_link_spec.rb | ||
generic_commit_status_spec.rb | ||
global_milestone_spec.rb | ||
group_spec.rb | ||
identity_spec.rb | ||
issue_spec.rb | ||
jira_issue_spec.rb | ||
key_spec.rb | ||
label_link_spec.rb | ||
label_spec.rb | ||
legacy_diff_note_spec.rb | ||
member_spec.rb | ||
merge_request_spec.rb | ||
milestone_spec.rb | ||
namespace_spec.rb | ||
note_spec.rb | ||
notification_setting_spec.rb | ||
project_group_link_spec.rb | ||
project_security_spec.rb | ||
project_snippet_spec.rb | ||
project_spec.rb | ||
project_team_spec.rb | ||
project_wiki_spec.rb | ||
protected_branch_spec.rb | ||
release_spec.rb | ||
repository_spec.rb | ||
service_spec.rb | ||
snippet_spec.rb | ||
spam_log_spec.rb | ||
todo_spec.rb | ||
tree_spec.rb | ||
user_spec.rb | ||
wiki_page_spec.rb |