diff options
author | GitLab Bot <gitlab-bot@gitlab.com> | 2021-12-20 16:37:47 +0300 |
---|---|---|
committer | GitLab Bot <gitlab-bot@gitlab.com> | 2021-12-20 16:37:47 +0300 |
commit | aee0a117a889461ce8ced6fcf73207fe017f1d99 (patch) | |
tree | 891d9ef189227a8445d83f35c1b0fc99573f4380 /doc/development/sql.md | |
parent | 8d46af3258650d305f53b819eabf7ab18d22f59e (diff) |
Add latest changes from gitlab-org/gitlab@14-6-stable-eev14.6.0-rc42
Diffstat (limited to 'doc/development/sql.md')
-rw-r--r-- | doc/development/sql.md | 3 |
1 files changed, 3 insertions, 0 deletions
diff --git a/doc/development/sql.md b/doc/development/sql.md index 129280598fe..e2208caf35a 100644 --- a/doc/development/sql.md +++ b/doc/development/sql.md @@ -249,6 +249,9 @@ In line with our `CodeReuse/ActiveRecord` cop, you should only use forms like use the `ApplicationRecord`-provided `.pluck_primary_key` helper method instead. In the latter, you should add a small helper method to the relevant model. +If you have strong reasons to use `pluck`, it could make sense to limit the number +of records plucked. `MAX_PLUCK` defaults to `1_000` in `ApplicationRecord`. + ## Inherit from ApplicationRecord Most models in the GitLab codebase should inherit from `ApplicationRecord`, |