Skip to content

Port to EE: Avoid plucking Todo ids and use sub-queries instead

Toon Claes requested to merge tc-no-todo-service-select-ee into master

What does this MR do?

TodoService should not call .select(&:id) on todos, because this is bad performance. So instead use sub-queries, which will result in a single SQL query to the database.

https://docs.gitlab.com/ee/development/sql.html#plucking-ids

Related

EE port of gitlab-org/gitlab-ce!10845

Merge request reports