Make Sidekiq transaction-aware to optimise posting runtime

danieljames-dj · Dec 2, 2023 · 96bfa65 · 96bfa65
1 parent 6949d9a
commit 96bfa65
Show file tree

Hide file tree

Showing 3 changed files with 8 additions and 0 deletions.
diff --git a/WcaOnRails/Gemfile b/WcaOnRails/Gemfile
@@ -103,6 +103,7 @@ gem 'sprockets-rails'
 gem 'fuzzy-string-match'
 gem 'sidekiq'
 gem 'sidekiq-cron'
+gem 'after_commit_everywhere'
 
 group :development, :test do
   gem 'spring'

diff --git a/WcaOnRails/Gemfile.lock b/WcaOnRails/Gemfile.lock
@@ -134,6 +134,9 @@ GEM
       tzinfo (~> 2.0)
     addressable (2.8.5)
       public_suffix (>= 2.0.2, < 6.0)
+    after_commit_everywhere (1.3.1)
+      activerecord (>= 4.2)
+      activesupport
     ansi (1.5.0)
     api-pagination (5.0.0)
     ast (2.4.2)
@@ -768,6 +771,7 @@ PLATFORMS
 DEPENDENCIES
   activerecord-import
   activestorage-validator
+  after_commit_everywhere
   api-pagination
   apparition!
   aws-sdk-cloudfront

diff --git a/WcaOnRails/config/initializers/sidekiq.rb b/WcaOnRails/config/initializers/sidekiq.rb
@@ -20,3 +20,6 @@
 Sidekiq.configure_client do |config|
   config.redis = { url: EnvConfig.SIDEKIQ_REDIS_URL }
 end
+
+# Make sure that job inserts during transaction are only inserted when the transaction completes.
+Sidekiq.transactional_push!