From cb913cf55f3f8ff07ad301e2f2c402af5fbb1d28 Mon Sep 17 00:00:00 2001 From: Max Wofford Date: Fri, 21 Feb 2025 20:32:03 -0500 Subject: [PATCH] Run robocop --- db/cable_schema.rb | 6 ++--- db/cache_schema.rb | 6 ++--- db/migrate/20250221224605_create_good_jobs.rb | 24 +++++++++---------- 3 files changed, 18 insertions(+), 18 deletions(-) diff --git a/db/cable_schema.rb b/db/cable_schema.rb index 2366660..eef9db1 100644 --- a/db/cable_schema.rb +++ b/db/cable_schema.rb @@ -4,8 +4,8 @@ ActiveRecord::Schema[7.1].define(version: 1) do t.binary "payload", limit: 536870912, null: false t.datetime "created_at", null: false t.integer "channel_hash", limit: 8, null: false - t.index ["channel"], name: "index_solid_cable_messages_on_channel" - t.index ["channel_hash"], name: "index_solid_cable_messages_on_channel_hash" - t.index ["created_at"], name: "index_solid_cable_messages_on_created_at" + t.index [ "channel" ], name: "index_solid_cable_messages_on_channel" + t.index [ "channel_hash" ], name: "index_solid_cable_messages_on_channel_hash" + t.index [ "created_at" ], name: "index_solid_cable_messages_on_created_at" end end diff --git a/db/cache_schema.rb b/db/cache_schema.rb index 6005a29..3ac1f3f 100644 --- a/db/cache_schema.rb +++ b/db/cache_schema.rb @@ -7,8 +7,8 @@ ActiveRecord::Schema[7.2].define(version: 1) do t.datetime "created_at", null: false t.integer "key_hash", limit: 8, null: false t.integer "byte_size", limit: 4, null: false - t.index ["byte_size"], name: "index_solid_cache_entries_on_byte_size" - t.index ["key_hash", "byte_size"], name: "index_solid_cache_entries_on_key_hash_and_byte_size" - t.index ["key_hash"], name: "index_solid_cache_entries_on_key_hash", unique: true + t.index [ "byte_size" ], name: "index_solid_cache_entries_on_byte_size" + t.index [ "key_hash", "byte_size" ], name: "index_solid_cache_entries_on_key_hash_and_byte_size" + t.index [ "key_hash" ], name: "index_solid_cache_entries_on_key_hash", unique: true end end diff --git a/db/migrate/20250221224605_create_good_jobs.rb b/db/migrate/20250221224605_create_good_jobs.rb index f64ec74..7415f8b 100644 --- a/db/migrate/20250221224605_create_good_jobs.rb +++ b/db/migrate/20250221224605_create_good_jobs.rb @@ -79,25 +79,25 @@ class CreateGoodJobs < ActiveRecord::Migration[8.0] end add_index :good_jobs, :scheduled_at, where: "(finished_at IS NULL)", name: :index_good_jobs_on_scheduled_at - add_index :good_jobs, [:queue_name, :scheduled_at], where: "(finished_at IS NULL)", name: :index_good_jobs_on_queue_name_and_scheduled_at - add_index :good_jobs, [:active_job_id, :created_at], name: :index_good_jobs_on_active_job_id_and_created_at + add_index :good_jobs, [ :queue_name, :scheduled_at ], where: "(finished_at IS NULL)", name: :index_good_jobs_on_queue_name_and_scheduled_at + add_index :good_jobs, [ :active_job_id, :created_at ], name: :index_good_jobs_on_active_job_id_and_created_at add_index :good_jobs, :concurrency_key, where: "(finished_at IS NULL)", name: :index_good_jobs_on_concurrency_key_when_unfinished - add_index :good_jobs, [:cron_key, :created_at], where: "(cron_key IS NOT NULL)", name: :index_good_jobs_on_cron_key_and_created_at_cond - add_index :good_jobs, [:cron_key, :cron_at], where: "(cron_key IS NOT NULL)", unique: true, name: :index_good_jobs_on_cron_key_and_cron_at_cond - add_index :good_jobs, [:finished_at], where: "retried_good_job_id IS NULL AND finished_at IS NOT NULL", name: :index_good_jobs_jobs_on_finished_at - add_index :good_jobs, [:priority, :created_at], order: { priority: "DESC NULLS LAST", created_at: :asc }, + add_index :good_jobs, [ :cron_key, :created_at ], where: "(cron_key IS NOT NULL)", name: :index_good_jobs_on_cron_key_and_created_at_cond + add_index :good_jobs, [ :cron_key, :cron_at ], where: "(cron_key IS NOT NULL)", unique: true, name: :index_good_jobs_on_cron_key_and_cron_at_cond + add_index :good_jobs, [ :finished_at ], where: "retried_good_job_id IS NULL AND finished_at IS NOT NULL", name: :index_good_jobs_jobs_on_finished_at + add_index :good_jobs, [ :priority, :created_at ], order: { priority: "DESC NULLS LAST", created_at: :asc }, where: "finished_at IS NULL", name: :index_good_jobs_jobs_on_priority_created_at_when_unfinished - add_index :good_jobs, [:priority, :created_at], order: { priority: "ASC NULLS LAST", created_at: :asc }, + add_index :good_jobs, [ :priority, :created_at ], order: { priority: "ASC NULLS LAST", created_at: :asc }, where: "finished_at IS NULL", name: :index_good_job_jobs_for_candidate_lookup - add_index :good_jobs, [:batch_id], where: "batch_id IS NOT NULL" - add_index :good_jobs, [:batch_callback_id], where: "batch_callback_id IS NOT NULL" + add_index :good_jobs, [ :batch_id ], where: "batch_id IS NOT NULL" + add_index :good_jobs, [ :batch_callback_id ], where: "batch_callback_id IS NOT NULL" add_index :good_jobs, :labels, using: :gin, where: "(labels IS NOT NULL)", name: :index_good_jobs_on_labels - add_index :good_job_executions, [:active_job_id, :created_at], name: :index_good_job_executions_on_active_job_id_and_created_at - add_index :good_jobs, [:priority, :scheduled_at], order: { priority: "ASC NULLS LAST", scheduled_at: :asc }, + add_index :good_job_executions, [ :active_job_id, :created_at ], name: :index_good_job_executions_on_active_job_id_and_created_at + add_index :good_jobs, [ :priority, :scheduled_at ], order: { priority: "ASC NULLS LAST", scheduled_at: :asc }, where: "finished_at IS NULL AND locked_by_id IS NULL", name: :index_good_jobs_on_priority_scheduled_at_unfinished_unlocked add_index :good_jobs, :locked_by_id, where: "locked_by_id IS NOT NULL", name: "index_good_jobs_on_locked_by_id" - add_index :good_job_executions, [:process_id, :created_at], name: :index_good_job_executions_on_process_id_and_created_at + add_index :good_job_executions, [ :process_id, :created_at ], name: :index_good_job_executions_on_process_id_and_created_at end end