Commit dfecec94 authored by Dugtuff's avatar Dugtuff Committed by Marcin Sedlak-Jakubowski

Fix the docs warning detected by the vale Acronym rule

parent efd99057
...@@ -47,9 +47,9 @@ This is more complicated than is ideal. It makes the query construction more ...@@ -47,9 +47,9 @@ This is more complicated than is ideal. It makes the query construction more
prone to errors (such as prone to errors (such as
[issue #15557](https://gitlab.com/gitlab-org/gitlab-foss/-/issues/15557)). [issue #15557](https://gitlab.com/gitlab-org/gitlab-foss/-/issues/15557)).
## Attempt A: WHERE EXISTS ## Attempt A: `WHERE EXISTS`
### Attempt A1: use multiple subqueries with WHERE EXISTS ### Attempt A1: use multiple subqueries with `WHERE EXISTS`
In [issue #37137](https://gitlab.com/gitlab-org/gitlab-foss/-/issues/37137) In [issue #37137](https://gitlab.com/gitlab-org/gitlab-foss/-/issues/37137)
and its associated [merge request](https://gitlab.com/gitlab-org/gitlab-foss/-/merge_requests/14022), and its associated [merge request](https://gitlab.com/gitlab-org/gitlab-foss/-/merge_requests/14022),
...@@ -82,7 +82,7 @@ AND (EXISTS ( ...@@ -82,7 +82,7 @@ AND (EXISTS (
While this worked without schema changes, and did improve readability somewhat, While this worked without schema changes, and did improve readability somewhat,
it did not improve query performance. it did not improve query performance.
### Attempt A2: use label IDs in the WHERE EXISTS clause ### Attempt A2: use label IDs in the `WHERE EXISTS` clause
In [merge request #34503](https://gitlab.com/gitlab-org/gitlab/-/merge_requests/34503), we followed a similar approach to A1. But this time, we In [merge request #34503](https://gitlab.com/gitlab-org/gitlab/-/merge_requests/34503), we followed a similar approach to A1. But this time, we
did a separate query to fetch the IDs of the labels used in the filter so that we avoid the `JOIN` in the `EXISTS` clause and filter directly by did a separate query to fetch the IDs of the labels used in the filter so that we avoid the `JOIN` in the `EXISTS` clause and filter directly by
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment