Skip to content

Remove extra todos exist query

Adam Hegyi requested to merge ah-reduce-query-count-on-todos-page into master

What does this MR do and why?

We check 3 times on the todos index page if the user has todos, every time we invoke an SQL query. This tiny change will now make only one existence check. In reality the 3 queries were not a big overhead because of the AR query cache.

Queries:

image

MR acceptance checklist

Please evaluate this MR against the MR acceptance checklist. It helps you analyze changes to reduce risks in quality, performance, reliability, security, and maintainability.

Edited by Adam Hegyi

Merge request reports