switchman-inst-jobs 1.5.2 → 3.0.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 99fc38ce3e38711c3ebb2d35f6e6eccb62f5d9388e6fbec9b9ba40830b3b4df7
-  data.tar.gz: 12501153af5a11a2c4d958cbe60d2185d684d92794081b1bbffe6f3394a6b257
+  metadata.gz: 711bf6fd7dfbea7baa7bf9f663264fdcfc37c3aa137d7b01f4c64985274dd045
+  data.tar.gz: 24955492cd97b95fbf3a8ab38c989fc40e0355751ddb43474c3c77b4839cdc43
 SHA512:
-  metadata.gz: 236773fe9bc45914fc1e78fe45ba8e16793259a93c2e58fe3a6603d04f03d4a748c8fd51171bbcb1440dd39e56a5b016bf9864668f540b8322f2b7f354ae68fa
-  data.tar.gz: 42d222e8d2660833010b731295cf76984e5e8cff9097c911260ab4ea0491771088685f73c0a9e05d335a38af79efdf34ad71296f7017297cdd9d382f8163f2e9
+  metadata.gz: 8e498902747984003acc07838e7cc596cd06c1bc0e9d9f88d24fb01c2eeef8d952e47f1da8913b4bea759cb19dde0e5ee0b0cb1651cb2c07e9708a34cef4ecaf
+  data.tar.gz: 547333f01e1ec2aad62221c76702c33ce07b67009f2b584e0187d152ded1c468637514c3e401c83389898a24386c6c72d74ffb1c7b2e7e5e6a3b50591a3ebfa6

data/db/migrate/20101216224513_create_delayed_jobs.rb

@@ -0,0 +1,42 @@
+class CreateDelayedJobs < ActiveRecord::Migration[4.2]
+  def connection
+    Delayed::Backend::ActiveRecord::Job.connection
+  end
+
+  def up
+    unless connection.adapter_name == 'PostgreSQL'
+      raise("#{connection.adapter_name} is not supported for delayed jobs queue")
+    end
+
+    create_table :delayed_jobs do |table|
+      # Allows some jobs to jump to the front of the queue
+      table.integer  :priority, default: 0
+      # Provides for retries, but still fail eventually.
+      table.integer  :attempts, default: 0
+      # YAML-encoded string of the object that will do work
+      table.text     :handler, limit: (500 * 1024)
+      # reason for last failure (See Note below)
+      table.text     :last_error
+      # The queue that this job is in
+      table.string   :queue, default: nil
+      # When to run.
+      # Could be Time.zone.now for immediately, or sometime in the future.
+      table.datetime :run_at
+      # Set when a client is working on this object
+      table.datetime :locked_at
+      # Set when all retries have failed
+      table.datetime :failed_at
+      # Who is working on this object (if locked)
+      table.string   :locked_by
+
+      table.timestamps
+    end
+
+    add_index :delayed_jobs, %i[priority run_at], name: 'delayed_jobs_priority'
+    add_index :delayed_jobs, [:queue], name: 'delayed_jobs_queue'
+  end
+
+  def down
+    drop_table :delayed_jobs
+  end
+end

data/db/migrate/20110208031356_add_delayed_jobs_tag.rb

@@ -0,0 +1,14 @@
+class AddDelayedJobsTag < ActiveRecord::Migration[4.2]
+  def connection
+    Delayed::Backend::ActiveRecord::Job.connection
+  end
+
+  def up
+    add_column :delayed_jobs, :tag, :string
+    add_index :delayed_jobs, [:tag]
+  end
+
+  def down
+    remove_column :delayed_jobs, :tag
+  end
+end

data/db/migrate/20110426161613_add_delayed_jobs_max_attempts.rb

@@ -0,0 +1,13 @@
+class AddDelayedJobsMaxAttempts < ActiveRecord::Migration[4.2]
+  def connection
+    Delayed::Backend::ActiveRecord::Job.connection
+  end
+
+  def up
+    add_column :delayed_jobs, :max_attempts, :integer
+  end
+
+  def down
+    remove_column :delayed_jobs, :max_attempts
+  end
+end

data/db/migrate/20110516225834_add_delayed_jobs_strand.rb

@@ -0,0 +1,14 @@
+class AddDelayedJobsStrand < ActiveRecord::Migration[4.2]
+  def connection
+    Delayed::Backend::ActiveRecord::Job.connection
+  end
+
+  def up
+    add_column :delayed_jobs, :strand, :string
+    add_index :delayed_jobs, :strand
+  end
+
+  def down
+    remove_column :delayed_jobs, :strand
+  end
+end

data/db/migrate/20110531144916_cleanup_delayed_jobs_indexes.rb

@@ -0,0 +1,26 @@
+class CleanupDelayedJobsIndexes < ActiveRecord::Migration[4.2]
+  def connection
+    Delayed::Backend::ActiveRecord::Job.connection
+  end
+
+  def up
+    case connection.adapter_name
+    when 'PostgreSQL'
+      # "nulls first" syntax is postgresql specific, and allows for more
+      # efficient querying for the next job
+      connection.execute("CREATE INDEX get_delayed_jobs_index ON #{::Delayed::Job.quoted_table_name} (priority, run_at, failed_at nulls first, locked_at nulls first, queue)")
+    else
+      add_index :delayed_jobs, %w[priority run_at locked_at failed_at queue], name: 'get_delayed_jobs_index'
+    end
+
+    # unused indexes
+    remove_index :delayed_jobs, name: 'delayed_jobs_queue'
+    remove_index :delayed_jobs, name: 'delayed_jobs_priority'
+  end
+
+  def down
+    remove_index :delayed_jobs, name: 'get_delayed_jobs_index'
+    add_index :delayed_jobs, %i[priority run_at], name: 'delayed_jobs_priority'
+    add_index :delayed_jobs, [:queue], name: 'delayed_jobs_queue'
+  end
+end

data/db/migrate/20110610213249_optimize_delayed_jobs.rb

@@ -0,0 +1,40 @@
+class OptimizeDelayedJobs < ActiveRecord::Migration[4.2]
+  def connection
+    Delayed::Backend::ActiveRecord::Job.connection
+  end
+
+  def up
+    create_table :failed_jobs do |t|
+      t.integer  'priority',    default: 0
+      t.integer  'attempts',    default: 0
+      t.string   'handler',     limit: 512_000
+      t.integer  'original_id', limit: 8
+      t.text     'last_error'
+      t.string   'queue'
+      t.datetime 'run_at'
+      t.datetime 'locked_at'
+      t.datetime 'failed_at'
+      t.string   'locked_by'
+      t.datetime 'created_at'
+      t.datetime 'updated_at'
+      t.string   'tag'
+      t.integer  'max_attempts'
+      t.string   'strand'
+    end
+
+    remove_index :delayed_jobs, name: 'get_delayed_jobs_index'
+    remove_index :delayed_jobs, [:strand]
+
+    add_index :delayed_jobs, %w[run_at queue locked_at strand priority], name: 'index_delayed_jobs_for_get_next'
+    add_index :delayed_jobs, %w[strand id], name: 'index_delayed_jobs_on_strand'
+
+    # move all failed jobs to the new failed table
+    Delayed::Backend::ActiveRecord::Job.where('failed_at IS NOT NULL').find_each do |job|
+      job.fail! unless job.on_hold?
+    end
+  end
+
+  def down
+    raise ActiveRecord::IrreversibleMigration
+  end
+end

data/db/migrate/20110831210257_add_delayed_jobs_next_in_strand.rb

@@ -0,0 +1,52 @@
+class AddDelayedJobsNextInStrand < ActiveRecord::Migration[4.2]
+  def connection
+    Delayed::Backend::ActiveRecord::Job.connection
+  end
+
+  def up
+    remove_index :delayed_jobs, name: 'index_delayed_jobs_for_get_next'
+
+    add_column :delayed_jobs, :next_in_strand, :boolean, default: true, null: false
+
+    # create the new index
+    connection.execute("CREATE INDEX get_delayed_jobs_index ON #{::Delayed::Job.quoted_table_name} (priority, run_at, queue) WHERE locked_at IS NULL AND next_in_strand = 't'")
+
+    # create the insert trigger
+    execute(<<-CODE)
+    CREATE FUNCTION #{connection.quote_table_name('delayed_jobs_before_insert_row_tr_fn')} () RETURNS trigger AS $$
+    BEGIN
+      LOCK delayed_jobs IN SHARE ROW EXCLUSIVE MODE;
+      IF (SELECT 1 FROM delayed_jobs WHERE strand = NEW.strand LIMIT 1) = 1 THEN
+        NEW.next_in_strand := 'f';
+      END IF;
+      RETURN NEW;
+    END;
+    $$ LANGUAGE plpgsql SET search_path TO #{::Switchman::Shard.current.name};
+    CODE
+    execute("CREATE TRIGGER delayed_jobs_before_insert_row_tr BEFORE INSERT ON #{::Delayed::Job.quoted_table_name} FOR EACH ROW WHEN (NEW.strand IS NOT NULL) EXECUTE PROCEDURE #{connection.quote_table_name('delayed_jobs_before_insert_row_tr_fn')}()")
+
+    # create the delete trigger
+    execute(<<-CODE)
+    CREATE FUNCTION #{connection.quote_table_name('delayed_jobs_after_delete_row_tr_fn')} () RETURNS trigger AS $$
+    BEGIN
+      UPDATE delayed_jobs SET next_in_strand = 't' WHERE id = (SELECT id FROM delayed_jobs j2 WHERE j2.strand = OLD.strand ORDER BY j2.strand, j2.id ASC LIMIT 1);
+      RETURN OLD;
+    END;
+    $$ LANGUAGE plpgsql SET search_path TO #{::Switchman::Shard.current.name};
+    CODE
+    execute("CREATE TRIGGER delayed_jobs_after_delete_row_tr AFTER DELETE ON #{::Delayed::Job.quoted_table_name} FOR EACH ROW WHEN (OLD.strand IS NOT NULL AND OLD.next_in_strand = 't') EXECUTE PROCEDURE #{connection.quote_table_name('delayed_jobs_after_delete_row_tr_fn')} ()")
+
+    execute(%{UPDATE #{::Delayed::Job.quoted_table_name} SET next_in_strand = 'f' WHERE strand IS NOT NULL AND id <> (SELECT id FROM #{::Delayed::Job.quoted_table_name} j2 WHERE j2.strand = delayed_jobs.strand ORDER BY j2.strand, j2.id ASC LIMIT 1)})
+  end
+
+  def down
+    execute %(DROP TRIGGER delayed_jobs_before_insert_row_tr ON #{::Delayed::Job.quoted_table_name})
+    execute %{DROP FUNCTION #{connection.quote_table_name('delayed_jobs_before_insert_row_tr_fn')} ()}
+    execute %(DROP TRIGGER delayed_jobs_after_delete_row_tr ON #{::Delayed::Job.quoted_table_name})
+    execute %{DROP FUNCTION #{connection.quote_table_name('delayed_jobs_after_delete_row_tr_fn')} ()}
+
+    remove_column :delayed_jobs, :next_in_strand
+    remove_index :delayed_jobs, name: 'get_delayed_jobs_index'
+    add_index :delayed_jobs, %w[run_at queue locked_at strand priority], name: 'index_delayed_jobs_for_get_next'
+  end
+end

data/db/migrate/20120510004759_delayed_jobs_delete_trigger_lock_for_update.rb

@@ -0,0 +1,31 @@
+class DelayedJobsDeleteTriggerLockForUpdate < ActiveRecord::Migration[4.2]
+  def connection
+    Delayed::Backend::ActiveRecord::Job.connection
+  end
+
+  def up
+    if connection.adapter_name == 'PostgreSQL'
+      execute(<<-CODE)
+      CREATE OR REPLACE FUNCTION #{connection.quote_table_name('delayed_jobs_after_delete_row_tr_fn')} () RETURNS trigger AS $$
+      BEGIN
+        UPDATE delayed_jobs SET next_in_strand = 't' WHERE id = (SELECT id FROM delayed_jobs j2 WHERE j2.strand = OLD.strand ORDER BY j2.strand, j2.id ASC LIMIT 1 FOR UPDATE);
+        RETURN OLD;
+      END;
+      $$ LANGUAGE plpgsql SET search_path TO #{::Switchman::Shard.current.name};
+      CODE
+    end
+  end
+
+  def down
+    if connection.adapter_name == 'PostgreSQL'
+      execute(<<-CODE)
+      CREATE OR REPLACE FUNCTION #{connection.quote_table_name('delayed_jobs_after_delete_row_tr_fn')} () RETURNS trigger AS $$
+      BEGIN
+        UPDATE delayed_jobs SET next_in_strand = 't' WHERE id = (SELECT id FROM delayed_jobs j2 WHERE j2.strand = OLD.strand ORDER BY j2.strand, j2.id ASC LIMIT 1);
+        RETURN OLD;
+      END;
+      $$ LANGUAGE plpgsql SET search_path TO #{::Switchman::Shard.current.name};
+      CODE
+    end
+  end
+end

data/db/migrate/20120531150712_drop_psql_jobs_pop_fn.rb

@@ -0,0 +1,15 @@
+class DropPsqlJobsPopFn < ActiveRecord::Migration[4.2]
+  def connection
+    Delayed::Backend::ActiveRecord::Job.connection
+  end
+
+  def up
+    if connection.adapter_name == 'PostgreSQL'
+      connection.execute('DROP FUNCTION IF EXISTS pop_from_delayed_jobs(varchar, varchar, integer, integer, timestamp without time zone)')
+    end
+  end
+
+  def down
+    raise ActiveRecord::IrreversibleMigration
+  end
+end

data/db/migrate/20120607164022_delayed_jobs_use_advisory_locks.rb

@@ -0,0 +1,80 @@
+class DelayedJobsUseAdvisoryLocks < ActiveRecord::Migration[4.2]
+  def connection
+    Delayed::Backend::ActiveRecord::Job.connection
+  end
+
+  def up
+    # use an advisory lock based on the name of the strand, instead of locking the whole table
+    # note that we're using half of the md5, so collisions are possible, but we don't really
+    # care because that would just be the old behavior, whereas for the most part locking will
+    # be much smaller
+    if connection.adapter_name == 'PostgreSQL'
+      execute(<<-CODE)
+      CREATE FUNCTION #{connection.quote_table_name('half_md5_as_bigint')}(strand varchar) RETURNS bigint AS $$
+      DECLARE
+        strand_md5 bytea;
+      BEGIN
+        strand_md5 := decode(md5(strand), 'hex');
+        RETURN (CAST(get_byte(strand_md5, 0) AS bigint) << 56) +
+                                  (CAST(get_byte(strand_md5, 1) AS bigint) << 48) +
+                                  (CAST(get_byte(strand_md5, 2) AS bigint) << 40) +
+                                  (CAST(get_byte(strand_md5, 3) AS bigint) << 32) +
+                                  (CAST(get_byte(strand_md5, 4) AS bigint) << 24) +
+                                  (get_byte(strand_md5, 5) << 16) +
+                                  (get_byte(strand_md5, 6) << 8) +
+                                   get_byte(strand_md5, 7);
+      END;
+      $$ LANGUAGE plpgsql SET search_path TO #{::Switchman::Shard.current.name};
+      CODE
+
+      execute(<<-CODE)
+      CREATE OR REPLACE FUNCTION #{connection.quote_table_name('delayed_jobs_before_insert_row_tr_fn')} () RETURNS trigger AS $$
+      BEGIN
+        PERFORM pg_advisory_xact_lock(half_md5_as_bigint(NEW.strand));
+        IF (SELECT 1 FROM delayed_jobs WHERE strand = NEW.strand LIMIT 1) = 1 THEN
+          NEW.next_in_strand := 'f';
+        END IF;
+        RETURN NEW;
+      END;
+      $$ LANGUAGE plpgsql SET search_path TO #{::Switchman::Shard.current.name};
+      CODE
+
+      execute(<<-CODE)
+      CREATE OR REPLACE FUNCTION #{connection.quote_table_name('delayed_jobs_after_delete_row_tr_fn')} () RETURNS trigger AS $$
+      BEGIN
+        PERFORM pg_advisory_xact_lock(half_md5_as_bigint(OLD.strand));
+        UPDATE delayed_jobs SET next_in_strand = 't' WHERE id = (SELECT id FROM delayed_jobs j2 WHERE j2.strand = OLD.strand ORDER BY j2.strand, j2.id ASC LIMIT 1 FOR UPDATE);
+        RETURN OLD;
+      END;
+      $$ LANGUAGE plpgsql SET search_path TO #{::Switchman::Shard.current.name};
+      CODE
+    end
+  end
+
+  def down
+    if connection.adapter_name == 'PostgreSQL'
+      execute(<<-CODE)
+      CREATE OR REPLACE FUNCTION #{connection.quote_table_name('delayed_jobs_before_insert_row_tr_fn')} () RETURNS trigger AS $$
+      BEGIN
+        LOCK delayed_jobs IN SHARE ROW EXCLUSIVE MODE;
+        IF (SELECT 1 FROM delayed_jobs WHERE strand = NEW.strand LIMIT 1) = 1 THEN
+          NEW.next_in_strand := 'f';
+        END IF;
+        RETURN NEW;
+      END;
+      $$ LANGUAGE plpgsql SET search_path TO #{::Switchman::Shard.current.name};
+      CODE
+
+      execute(<<-CODE)
+      CREATE OR REPLACE FUNCTION #{connection.quote_table_name('delayed_jobs_after_delete_row_tr_fn')} () RETURNS trigger AS $$
+      BEGIN
+        UPDATE delayed_jobs SET next_in_strand = 't' WHERE id = (SELECT id FROM delayed_jobs j2 WHERE j2.strand = OLD.strand ORDER BY j2.strand, j2.id ASC LIMIT 1 FOR UPDATE);
+        RETURN OLD;
+      END;
+      $$ LANGUAGE plpgsql SET search_path TO #{::Switchman::Shard.current.name};
+      CODE
+
+      execute("DROP FUNCTION #{connection.quote_table_name('half_md5_as_bigint')}(varchar)")
+    end
+  end
+end

data/db/migrate/20120607181141_index_jobs_on_locked_by.rb

@@ -0,0 +1,15 @@
+class IndexJobsOnLockedBy < ActiveRecord::Migration[4.2]
+  disable_ddl_transaction! if respond_to?(:disable_ddl_transaction!)
+
+  def connection
+    Delayed::Backend::ActiveRecord::Job.connection
+  end
+
+  def up
+    add_index :delayed_jobs, :locked_by, algorithm: :concurrently, where: 'locked_by IS NOT NULL'
+  end
+
+  def down
+    remove_index :delayed_jobs, :locked_by
+  end
+end

data/db/migrate/20120608191051_add_jobs_run_at_index.rb

@@ -0,0 +1,15 @@
+class AddJobsRunAtIndex < ActiveRecord::Migration[4.2]
+  disable_ddl_transaction! if respond_to?(:disable_ddl_transaction!)
+
+  def connection
+    Delayed::Backend::ActiveRecord::Job.connection
+  end
+
+  def up
+    add_index :delayed_jobs, %w[run_at tag], algorithm: :concurrently
+  end
+
+  def down
+    remove_index :delayed_jobs, name: 'index_delayed_jobs_on_run_at_and_tag'
+  end
+end

data/db/migrate/20120927184213_change_delayed_jobs_handler_to_text.rb

@@ -0,0 +1,13 @@
+class ChangeDelayedJobsHandlerToText < ActiveRecord::Migration[4.2]
+  def connection
+    Delayed::Job.connection
+  end
+
+  def up
+    change_column :delayed_jobs, :handler, :text
+  end
+
+  def down
+    change_column :delayed_jobs, :handler, :string, limit: 500.kilobytes
+  end
+end

data/db/migrate/20140505215131_add_failed_jobs_original_job_id.rb

@@ -0,0 +1,13 @@
+class AddFailedJobsOriginalJobId < ActiveRecord::Migration[4.2]
+  def connection
+    Delayed::Backend::ActiveRecord::Job.connection
+  end
+
+  def up
+    add_column :failed_jobs, :original_job_id, :integer, limit: 8
+  end
+
+  def down
+    remove_column :failed_jobs, :original_job_id
+  end
+end

data/db/migrate/20140505215510_copy_failed_jobs_original_id.rb

@@ -0,0 +1,12 @@
+class CopyFailedJobsOriginalId < ActiveRecord::Migration[4.2]
+  def connection
+    Delayed::Backend::ActiveRecord::Job.connection
+  end
+
+  def up
+    # this is a smaller, less frequently accessed table, so we just update all at once
+    Delayed::Backend::ActiveRecord::Job::Failed.where('original_job_id is null').update_all('original_job_id = original_id')
+  end
+
+  def down; end
+end

data/db/migrate/20140505223637_drop_failed_jobs_original_id.rb

@@ -0,0 +1,13 @@
+class DropFailedJobsOriginalId < ActiveRecord::Migration[4.2]
+  def connection
+    Delayed::Backend::ActiveRecord::Job.connection
+  end
+
+  def up
+    remove_column :failed_jobs, :original_id
+  end
+
+  def down
+    add_column :failed_jobs, :original_id, :integer, limit: 8
+  end
+end