From 7d31849057954b768a6be4fee9ed8e8141b3ea88 Mon Sep 17 00:00:00 2001 From: ilya Date: Thu, 27 Feb 2025 12:49:49 +0000 Subject: [PATCH] [EASY] Reminder: Create indexes concurrently (#3298) Updates the DB changes reminder to consider using `CREATE INDEX CONCURRENTLY` for the tables involved in the critical execution path. --- .github/nitpicks.yml | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/.github/nitpicks.yml b/.github/nitpicks.yml index 0eab7e12b9..7b12de0eee 100644 --- a/.github/nitpicks.yml +++ b/.github/nitpicks.yml @@ -1,6 +1,7 @@ - markdown: | Reminder: Please update the DB Readme and comment whether migrations are reversible (include rollback scripts if applicable). - If new tables were added, update the [tables list](https://github.com/cowprotocol/services/blob/main/crates/database/src/lib.rs#L51-L87). + If creating new tables, update the [tables list](https://github.com/cowprotocol/services/blob/main/crates/database/src/lib.rs#L51-L87). + When adding a new index, consider using `CREATE INDEX CONCURRENTLY` for tables involved in the critical execution path. pathFilter: - "database/sql/**"