canvas_sync 0.17.23.beta5 → 0.17.23

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: ff0bdfef2b2ae070b727fefe7b16c20149a82e8237a922102516de8aaee508ae
-  data.tar.gz: da0f508fe25b22b3a9913d81891cf729effaf55d0d0800001079015101be1179
+  metadata.gz: 1e0b64f56f9bd68aa468f8bd608000f622c07a0dbcf1a49469435788e5708545
+  data.tar.gz: 48bd87ac7843bf65f5e4ac11bc5271a96a032244edcb2996e710006ecf007801
 SHA512:
-  metadata.gz: 48c84c48103000a58b3fd5803c650ccc3bf485c39d8cb09467892de0621d2607757542ed9dd275a741040231639d269978f2e5fe23d7830649470fb409bd2fd6
-  data.tar.gz: 36cd14b22dc554ecb7110770e4cb13ce9e0ec7cc1f000ee6b567d65493f07b1c616ec0b2ac3b7234a8ba3bf633f7468a709c10ae771183ae31237ad726476826
+  metadata.gz: 37835e33df63bbca5a712535b1fa32574e7f6d80a186cdb1e1638b5232ccb8f96cf36c95ae10483e7e519a0ddfee7334f91cd323757302da0304d15205274a62
+  data.tar.gz: 785bc9a767cad6b750c23dc16f7f5fec58e88c5fb8defe7156e7931c31ee5093de25c864aa9c4b2dd6aa72f884fbbc28fcd707dfda59a8f048d551c1a8c0d361

data/config/initializers/apartment.rb

@@ -7,14 +7,13 @@
 Rails.application.config.after_initialize do
   next unless defined?(Apartment)
 
-  # Apartment already solves this issue (and in a better way) in newer versions
+  # Newer versions of Apartment already solves this issue (and in a better way)
   begin
     require('apartment/version')
     next if Gem::Version.new(Apartment::VERSION) >= Gem::Version.new('2.8.1')
   rescue LoadError
   end
 
-
   begin
     Rails.application.eager_load!
     ActiveRecord::Base.descendants.each do |model|

data/lib/canvas_sync/generators/templates/migrations/create_grading_period_groups.rb

@@ -4,8 +4,6 @@ class CreateGradingPeriodGroups < ActiveRecord::Migration[5.1]
   def change
     create_table :grading_period_groups do |t|
       t.bigint :canvas_id, null: false
-      t.bigint :canvas_course_id
-      t.bigint :canvas_account_id
       t.string :title
       t.boolean :weighted
       t.boolean :display_totals_for_all_grading_periods
@@ -16,7 +14,5 @@ class CreateGradingPeriodGroups < ActiveRecord::Migration[5.1]
     end
 
     add_index :grading_period_groups, :canvas_id, unique: true
-    add_index :grading_period_groups, :canvas_course_id
-    add_index :grading_period_groups, :canvas_account_id
   end
 end

data/lib/canvas_sync/generators/templates/models/grading_period_group.rb

@@ -4,6 +4,4 @@ class GradingPeriodGroup < ApplicationRecord
   include CanvasSync::Record
 
   validates :canvas_id, uniqueness: true, presence: true
-  belongs_to :course, primary_key: :canvas_id, foreign_key: :canvas_course_id, optional: true
-  belongs_to :account, primary_key: :canvas_id, foreign_key: :canvas_account_id, optional: true
 end

data/lib/canvas_sync/job_batches/batch.rb

@@ -76,14 +76,12 @@ module CanvasSync
       def on(event, callback, options = {})
         return unless Callback::VALID_CALLBACKS.include?(event.to_s)
         callback_key = "#{@bidkey}-callbacks-#{event}"
-        redis do |r|
-          r.multi do
-            r.sadd(callback_key, JSON.unparse({
-              callback: callback,
-              opts: options
-            }))
-            r.expire(callback_key, BID_EXPIRE_TTL)
-          end
+        redis.multi do |r|
+          r.sadd(callback_key, JSON.unparse({
+            callback: callback,
+            opts: options
+          }))
+          r.expire(callback_key, BID_EXPIRE_TTL)
         end
       end
 
@@ -93,16 +91,14 @@ module CanvasSync
         if !@existing && !@initialized
           parent_bid = Thread.current[:batch]&.bid
 
-          redis do |r|
-            r.multi do
-              r.hset(@bidkey, "parent_bid", parent_bid.to_s) if parent_bid
-              r.expire(@bidkey, BID_EXPIRE_TTL)
+          redis.multi do |r|
+            r.hset(@bidkey, "parent_bid", parent_bid.to_s) if parent_bid
+            r.expire(@bidkey, BID_EXPIRE_TTL)
 
-              if parent_bid
-                r.hincrby("BID-#{parent_bid}", "children", 1)
-                r.expire("BID-#{parent_bid}", BID_EXPIRE_TTL)
-                r.zadd("BID-#{parent_bid}-bids", created_at, bid)
-              end
+            if parent_bid
+              r.hincrby("BID-#{parent_bid}", "children", 1)
+              r.expire("BID-#{parent_bid}", BID_EXPIRE_TTL)
+              r.zadd("BID-#{parent_bid}-bids", created_at, bid)
             end
           end
 
@@ -131,15 +127,11 @@ module CanvasSync
       end
 
       def invalidate_all
-        redis do |r|
-          r.setex("invalidated-bid-#{bid}", BID_EXPIRE_TTL, 1)
-        end
+        redis.setex("invalidated-bid-#{bid}", BID_EXPIRE_TTL, 1)
       end
 
       def parent_bid
-        redis do |r|
-          r.hget(@bidkey, "parent_bid")
-        end
+        redis.hget(@bidkey, "parent_bid")
       end
 
       def parent
@@ -149,7 +141,7 @@ module CanvasSync
       end
 
       def valid?(batch = self)
-        valid = !redis { |r| r.exists?("invalidated-bid-#{batch.bid}") }
+        valid = !redis.exists?("invalidated-bid-#{batch.bid}")
         batch.parent ? valid && valid?(batch.parent) : valid
       end
 
@@ -162,23 +154,19 @@ module CanvasSync
             let_close!
           end
         else
-          redis do |r|
-            r.hset(@bidkey, 'keep_open', true)
-          end
+          redis.hset(@bidkey, 'keep_open', true)
         end
       end
 
       def let_close!
-        _, failed, pending, children, complete, success = redis do |r|
-          r.multi do
-            r.hset(@bidkey, 'keep_open', false)
-
-            r.scard("BID-#{bid}-failed")
-            r.hincrby("BID-#{bid}", "pending", 0)
-            r.hincrby("BID-#{bid}", "children", 0)
-            r.scard("BID-#{bid}-batches-complete")
-            r.scard("BID-#{bid}-batches-success")
-          end
+        _, failed, pending, children, complete, success = redis.multi do |r|
+          r.hset(@bidkey, 'keep_open', false)
+
+          r.scard("BID-#{bid}-failed")
+          r.hincrby("BID-#{bid}", "pending", 0)
+          r.hincrby("BID-#{bid}", "children", 0)
+          r.scard("BID-#{bid}-batches-complete")
+          r.scard("BID-#{bid}-batches-success")
         end
 
         all_success = pending.to_i.zero? && children == success
@@ -211,18 +199,14 @@ module CanvasSync
 
       def flush_pending_attrs
         super
-        redis do |r|
-          r.zadd("batches", created_at, bid)
-        end
+        redis.zadd("batches", created_at, bid)
       end
 
       private
 
       def assert_batch_is_open
         unless defined?(@closed)
-          redis do |r|
-            @closed = r.hget(@bidkey, 'success') == 'true'
-          end
+          @closed = redis.hget(@bidkey, 'success') == 'true'
         end
         raise "Cannot add jobs to Batch #{} bid - it has already entered the callback-stage" if @closed
       end
@@ -234,7 +218,7 @@ module CanvasSync
         redis do |r|
           tme = Time.now.utc.to_f
           added = r.zadd(@bidkey + "-jids", jids.map{|jid| [tme, jid] }, nx: true)
-          r.multi do
+          r.multi do |r|
             r.hincrby(@bidkey, "pending", added)
             r.hincrby(@bidkey, "job_count", added)
             r.expire(@bidkey, BID_EXPIRE_TTL)
@@ -283,11 +267,9 @@ module CanvasSync
           end
 
           if parent_bid
-            redis do |r|
-              r.multi do
-                r.sadd("BID-#{parent_bid}-dead", jid)
-                r.expire("BID-#{parent_bid}-dead", BID_EXPIRE_TTL)
-              end
+            redis.multi do
+              r.sadd("BID-#{parent_bid}-dead", jid)
+              r.expire("BID-#{parent_bid}-dead", BID_EXPIRE_TTL)
             end
           end
 
@@ -413,8 +395,21 @@ module CanvasSync
           cleanup_redis(bid)
         end
 
-        def redis(*args, &blk)
-          defined?(::Sidekiq) ? ::Sidekiq.redis(*args, &blk) : nil # TODO
+        def redis(&blk)
+          return RedisProxy.new unless block_given?
+
+          if Thread.current[:job_batches_redis]
+            yield Thread.current[:job_batches_redis]
+          elsif defined?(::Sidekiq)
+            ::Sidekiq.redis do |r|
+              Thread.current[:job_batches_redis] = r
+              yield r
+            ensure
+              Thread.current[:job_batches_redis] = nil
+            end
+          else
+            # TODO
+          end
         end
 
         def logger
@@ -433,6 +428,42 @@ module CanvasSync
           end
         end
       end
+
+      class RedisProxy
+        def multi(*args, &block)
+          Batch.redis do |r|
+            if block.arity == 1
+              r.multi(*args) do
+                block.call(r)
+              end
+            else
+              r.multi(*args, &block)
+            end
+          end
+        end
+
+        def pipelined(*args, &block)
+          Batch.redis do |r|
+            if block.arity == 1
+              r.pipelined(*args) do
+                block.call(r)
+              end
+            else
+              r.pipelined(*args, &block)
+            end
+          end
+        end
+
+        def method_missing(method_name, *arguments, &block)
+          Batch.redis do |r|
+            r.send(method_name, *arguments, &block)
+          end
+        end
+
+        def respond_to_missing?(method_name, include_private = false)
+          super || Redis.method_defined?(method_name)
+        end
+      end
     end
 
     ActiveJob::Base.include BatchAwareJob

data/lib/canvas_sync/job_batches/pool.rb

@@ -64,18 +64,14 @@ module CanvasSync
             let_close!
           end
         else
-          redis do |r|
-            r.hset(redis_key, 'keep_open', true)
-          end
+          redis.hset(redis_key, 'keep_open', true)
         end
       end
 
       def let_close!
-        _, active_count = redis do |r|
-          r.multi do
-            r.hset(redis_key, 'keep_open', false)
-            r.hincrby(redis_key, "active_count", 0)
-          end
+        _, active_count = redis.multi do |r|
+          r.hset(redis_key, 'keep_open', false)
+          r.hincrby(redis_key, "active_count", 0)
         end
 
         if active_count == 0 && pending_count == 0
@@ -95,35 +91,38 @@ module CanvasSync
       end
 
       def active_count
-        redis do |r|
-          r.hincrby(redis_key, "active_count", 0)
-        end
+        redis.hincrby(redis_key, "active_count", 0)
       end
 
       def pending_count
         jobs_key = "#{redis_key}-jobs"
         order = self.order || 'fifo'
-        redis do |r|
-          case order.to_sym
-          when :fifo, :lifo
-            r.llen(jobs_key)
-          when :random
-            r.scard(jobs_key)
-          when :priority
-            r.zcard(jobs_key)
-          end
+        case order.to_sym
+        when :fifo, :lifo
+          redis.llen(jobs_key)
+        when :random
+          redis.scard(jobs_key)
+        when :priority
+          redis.zcard(jobs_key)
         end
       end
 
       def job_checked_in(status, options)
-        active_count = redis do |r|
+        active_count, pending_count = redis do |r|
           return unless r.exists?(redis_key)
-          r.hincrby(redis_key, "active_count", -1)
+
+          # Make sure this is loaded outside of the pipeline
+          self.order
+
+          redis.multi do
+            r.hincrby(redis_key, "active_count", -1)
+            self.pending_count
+          end
         end
 
         added_count = refill_allotment
-        if active_count == 0 && added_count == 0
-          if clean_when_empty && redis {|r| r.hget(redis_key, 'keep_open') } != 'true'
+        if active_count == 0 && added_count == 0 && pending_count == 0
+          if clean_when_empty && redis.hget(redis_key, 'keep_open') != 'true'
             cleanup_redis
           end
         end
@@ -175,18 +174,16 @@ module CanvasSync
         job_json = JSON.unparse(ActiveJob::Arguments.serialize([job_desc]))
         order = self.order
 
-        redis do |r|
-          r.multi do
-            case order.to_sym
-            when :fifo, :lifo
-              r.rpush(jobs_key, job_json)
-            when :random
-              r.sadd(jobs_key, job_json)
-            when :priority
-              r.zadd(jobs_key, job_desc[:priority] || 0, job_json)
-            end
-            r.expire(jobs_key, Batch::BID_EXPIRE_TTL)
+        redis.multi do |r|
+          case order.to_sym
+          when :fifo, :lifo
+            r.rpush(jobs_key, job_json)
+          when :random
+            r.sadd(jobs_key, job_json)
+          when :priority
+            r.zadd(jobs_key, job_desc[:priority] || 0, job_json)
           end
+          r.expire(jobs_key, Batch::BID_EXPIRE_TTL)
         end
       end
 
@@ -194,18 +191,15 @@ module CanvasSync
         jobs_key = "#{redis_key}-jobs"
         order = self.order
 
-        job_json = nil
-        redis do |r|
-          job_json = case order.to_sym
-          when :fifo
-            r.lpop(jobs_key)
-          when :lifo
-            r.rpop(jobs_key)
-          when :random
-            r.spop(jobs_key)
-          when :priority
-            r.zpopmax(jobs_key)
-          end
+        job_json = case order.to_sym
+        when :fifo
+          redis.lpop(jobs_key)
+        when :lifo
+          redis.rpop(jobs_key)
+        when :random
+          redis.spop(jobs_key)
+        when :priority
+          redis.zpopmax(jobs_key)
         end
 
         return nil unless job_json.present?
@@ -218,13 +212,9 @@ module CanvasSync
       end
       delegate :redis, to: :class
 
-      protected
-
       def flush_pending_attrs
         super
-        redis do |r|
-          r.zadd("pools", created_at, pid)
-        end
+        redis.zadd("pools", created_at, pid)
       end
 
       private

data/lib/canvas_sync/jobs/sync_terms_job.rb

@@ -9,7 +9,7 @@ module CanvasSync
       #   will be started for each term in that scope. :models should be an array of
       #   models to sync.
       def perform(options)
-        CanvasSync.get_canvas_sync_client(batch_context).terms("self").all_pages!.each do |term_params|
+        CanvasSync.get_canvas_sync_client(batch_context).terms("self", workflow_state: ['all']).all_pages!.each do |term_params|
           if account_id = batch_context[:account_id]
             # These branches are primarily to support Legacy apps
             if Term.respond_to?(:create_or_update) && Term.method(:create_or_update).arity.abs == 2

data/lib/canvas_sync/processors/model_mappings.yml

@@ -457,12 +457,6 @@ grading_period_groups:
     grading_period_group_id:
       database_column_name: canvas_id
       type: integer
-    canvas_course_id:
-      database_column_name: canvas_course_id
-      type: integer
-    canvas_account_id:
-      database_column_name: canvas_account_id
-      type: integer
     title:
       database_column_name: title
       type: string

data/lib/canvas_sync/version.rb

@@ -1,3 +1,3 @@
 module CanvasSync
-  VERSION = "0.17.23.beta5".freeze
+  VERSION = "0.17.23".freeze
 end

data/spec/dummy/app/models/grading_period_group.rb

@@ -10,6 +10,4 @@ class GradingPeriodGroup < ApplicationRecord
   include CanvasSync::Record
 
   validates :canvas_id, uniqueness: true, presence: true
-  belongs_to :course, primary_key: :canvas_id, foreign_key: :canvas_course_id, optional: true
-  belongs_to :account, primary_key: :canvas_id, foreign_key: :canvas_account_id, optional: true
 end

data/spec/dummy/app/models/term.rb

@@ -12,6 +12,7 @@ class Term < ApplicationRecord
 
   validates :canvas_id, uniqueness: true, presence: true
   has_many :courses, foreign_key: :canvas_term_id, primary_key: :canvas_id
+  belongs_to :grading_period_group, primary_key: :canvas_id, foreign_key: :grading_period_group_id, optional: true
 
   api_syncable({
     canvas_id: :id,

data/spec/dummy/db/migrate/{20210907233331_create_grading_period_groups.rb → 20211001184920_create_grading_period_groups.rb}

@@ -10,8 +10,6 @@ class CreateGradingPeriodGroups < ActiveRecord::Migration[5.1]
   def change
     create_table :grading_period_groups do |t|
       t.bigint :canvas_id, null: false
-      t.bigint :canvas_course_id
-      t.bigint :canvas_account_id
       t.string :title
       t.boolean :weighted
       t.boolean :display_totals_for_all_grading_periods
@@ -22,7 +20,5 @@ class CreateGradingPeriodGroups < ActiveRecord::Migration[5.1]
     end
 
     add_index :grading_period_groups, :canvas_id, unique: true
-    add_index :grading_period_groups, :canvas_course_id
-    add_index :grading_period_groups, :canvas_account_id
   end
 end

data/spec/dummy/db/schema.rb

@@ -10,7 +10,7 @@
 #
 # It's strongly recommended that you check this file into your version control system.
 
-ActiveRecord::Schema.define(version: 2021_09_07_233331) do
+ActiveRecord::Schema.define(version: 2021_10_01_184920) do
 
   # These are extensions that must be enabled in order to support this database
   enable_extension "plpgsql"
@@ -175,16 +175,12 @@ ActiveRecord::Schema.define(version: 2021_09_07_233331) do
 
   create_table "grading_period_groups", force: :cascade do |t|
     t.bigint "canvas_id", null: false
-    t.bigint "canvas_course_id"
-    t.bigint "canvas_account_id"
     t.string "title"
     t.boolean "weighted"
     t.boolean "display_totals_for_all_grading_periods"
     t.string "workflow_state"
     t.datetime "created_at", null: false
     t.datetime "updated_at", null: false
-    t.index ["canvas_account_id"], name: "index_grading_period_groups_on_canvas_account_id"
-    t.index ["canvas_course_id"], name: "index_grading_period_groups_on_canvas_course_id"
     t.index ["canvas_id"], name: "index_grading_period_groups_on_canvas_id", unique: true
   end