chdsbd / kodiak

🔮 A bot to automatically update and merge GitHub PRs
https://kodiakhq.com
GNU Affero General Public License v3.0
1.03k stars 65 forks source link

update(web_api): index more created_at based queries #794

Closed sbdchd closed 2 years ago

sbdchd commented 2 years ago

technically we only need to index:

but it was easier to update the parent class instead of the specific ones.

squawk-squawk[bot] commented 2 years ago

Squawk Report

🚒 8 violations across 1 file(s)


web_api/migrations/0026_auto_20220322_0036.sql

BEGIN;
--
-- Alter field created_at on account
--
CREATE INDEX "account_created_at_c71fc1ad" ON "account" ("created_at");
--
-- Alter field created_at on accountmembership
--
CREATE INDEX "account_membership_created_at_3c268f6c" ON "account_membership" ("created_at");
--
-- Alter field created_at on githubevent
--
CREATE INDEX "github_event_created_at_3f85f8ad" ON "github_event" ("created_at");
--
-- Alter field created_at on pullrequestactivity
--
CREATE INDEX "pull_request_activity_created_at_02396ea3" ON "pull_request_activity" ("created_at");
--
-- Alter field created_at on pullrequestactivityprogress
--
CREATE INDEX "pull_request_activity_progress_created_at_b795bfd0" ON "pull_request_activity_progress" ("created_at");
--
-- Alter field created_at on user
--
CREATE INDEX "user_created_at_64c716d1" ON "user" ("created_at");
--
-- Alter field created_at on userpullrequestactivity
--
CREATE INDEX "user_pull_request_activity_created_at_b93cdc90" ON "user_pull_request_activity" ("created_at");
--
-- Alter field created_at on userpullrequestactivityprogress
--
CREATE INDEX "user_pull_request_activity_progress_created_at_39c278b5" ON "user_pull_request_activity_progress" ("created_at");
COMMIT;

🚒 Rule Violations (8)

web_api/migrations/0026_auto_20220322_0036.sql:2:1: warning: require-concurrent-index-creation

   2 | --
   3 | -- Alter field created_at on account
   4 | --
   5 | CREATE INDEX "account_created_at_c71fc1ad" ON "account" ("created_at");

  note: Creating an index blocks writes.
  help: Create the index CONCURRENTLY.

web_api/migrations/0026_auto_20220322_0036.sql:6:1: warning: require-concurrent-index-creation

   6 | --
   7 | -- Alter field created_at on accountmembership
   8 | --
   9 | CREATE INDEX "account_membership_created_at_3c268f6c" ON "account_membership" ("created_at");

  note: Creating an index blocks writes.
  help: Create the index CONCURRENTLY.

web_api/migrations/0026_auto_20220322_0036.sql:10:1: warning: require-concurrent-index-creation

  10 | --
  11 | -- Alter field created_at on githubevent
  12 | --
  13 | CREATE INDEX "github_event_created_at_3f85f8ad" ON "github_event" ("created_at");

  note: Creating an index blocks writes.
  help: Create the index CONCURRENTLY.

web_api/migrations/0026_auto_20220322_0036.sql:14:1: warning: require-concurrent-index-creation

  14 | --
  15 | -- Alter field created_at on pullrequestactivity
  16 | --
  17 | CREATE INDEX "pull_request_activity_created_at_02396ea3" ON "pull_request_activity" ("created_at");

  note: Creating an index blocks writes.
  help: Create the index CONCURRENTLY.

web_api/migrations/0026_auto_20220322_0036.sql:18:1: warning: require-concurrent-index-creation

  18 | --
  19 | -- Alter field created_at on pullrequestactivityprogress
  20 | --
  21 | CREATE INDEX "pull_request_activity_progress_created_at_b795bfd0" ON "pull_request_activity_progress" ("created_at");

  note: Creating an index blocks writes.
  help: Create the index CONCURRENTLY.

web_api/migrations/0026_auto_20220322_0036.sql:22:1: warning: require-concurrent-index-creation

  22 | --
  23 | -- Alter field created_at on user
  24 | --
  25 | CREATE INDEX "user_created_at_64c716d1" ON "user" ("created_at");

  note: Creating an index blocks writes.
  help: Create the index CONCURRENTLY.

web_api/migrations/0026_auto_20220322_0036.sql:26:1: warning: require-concurrent-index-creation

  26 | --
  27 | -- Alter field created_at on userpullrequestactivity
  28 | --
  29 | CREATE INDEX "user_pull_request_activity_created_at_b93cdc90" ON "user_pull_request_activity" ("created_at");

  note: Creating an index blocks writes.
  help: Create the index CONCURRENTLY.

web_api/migrations/0026_auto_20220322_0036.sql:30:1: warning: require-concurrent-index-creation

  30 | --
  31 | -- Alter field created_at on userpullrequestactivityprogress
  32 | --
  33 | CREATE INDEX "user_pull_request_activity_progress_created_at_39c278b5" ON "user_pull_request_activity_progress" ("created_at");

  note: Creating an index blocks writes.
  help: Create the index CONCURRENTLY.

📚 More info on rules

⚡️ Powered by Squawk, a linter for PostgreSQL, focused on migrations