saviour 0.6.6 → 0.6.7

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: e01220554bae6d136bace3b491f366aa5e5f8a8e6b853b1d19f5fe2ee194f7ac
-  data.tar.gz: cce3bf80da5aad0f7056422f9e30d9595f5c2b597f899346f09f0b615e0c5ce3
+  metadata.gz: 43509e7e421eab9a8b45159d62c58513dbcb894b0ccecbd96a70cc0a29db74a6
+  data.tar.gz: 91b2eb4cf47dbac1b642c5ccc198ae0040ea0e81ef993c580e894065f1e58ecf
 SHA512:
-  metadata.gz: 0b9b2682ffb7d3584853b3cb3ceca37cbae5f95a9802eabbe5d3df80a5de79810dbb3f26e5cb29fdfcf2acb128a9cd8d9f459458e0dc43bf93c47ec1274af303
-  data.tar.gz: 0a94d1526dcbb231bb27feb1b47cd913dd668ee127e6ce41961a67d2cd14ff646a7f52b856d5dc2b2758c1e3ca59b12473f2a8b9227c5c1078d17cd860a90bbe
+  metadata.gz: e4a1fb654e3290d827236f115ebfd93e2f6d4994e57d022dd473f41436365de36182e689d20526cea2c9bb113dd6b558901da5deda945f526242ab2e78e7c633
+  data.tar.gz: 8f3f81305ceb56195d67eb2a6700a5f05bb3e4567c892a983fe845689c952864497e2bd29c47a4256a88a6f003527855952d2ed78e6edc7cff80d66adacca7b0

data/.travis.yml

@@ -2,8 +2,9 @@ language: ruby
 sudo: false
 cache: bundler
 rvm:
-  - 2.5.0
-  - 2.6.0
+  - 2.5.8
+  - 2.6.6
+  - 2.7.1
 
 gemfile:
   - gemfiles/5.1.gemfile

data/lib/saviour.rb

@@ -17,7 +17,8 @@ require 'saviour/db_helpers'
 
 require 'tempfile'
 require 'fileutils'
-require 'concurrent/future'
+require 'concurrent/edge/throttle'
+require 'concurrent/edge/lock_free_queue'
 
 require 'active_support/dependencies'
 

data/lib/saviour/life_cycle.rb

@@ -1,5 +1,7 @@
 module Saviour
   class LifeCycle
+    SHOULD_USE_INTERLOCK = defined?(Rails)
+
     class FileCreator
       def initialize(current_path, file, column, connection)
         @file = file
@@ -84,26 +86,19 @@ module Saviour
 
     def delete!
       DbHelpers.run_after_commit do
-        pool = Concurrent::FixedThreadPool.new(Saviour::Config.concurrent_workers)
+        pool = Concurrent::Throttle.new Saviour::Config.concurrent_workers
 
         futures = attached_files.map do |column|
-          Concurrent::Future.execute(executor: pool) {
-            file = @model.send(column)
+          pool.future(@model.send(column)) do |file|
             path = file.persisted_path
             file.uploader.storage.delete(path) if path
             file.delete
-          }
+          end
         end
 
         ActiveSupport::Dependencies.interlock.permit_concurrent_loads do
-          futures.each do |future|
-            future.value
-            raise(future.reason) if future.rejected?
-          end
+          futures.each(&:value!)
         end
-
-        pool.shutdown
-        pool.wait_for_termination
       end
     end
 
@@ -131,35 +126,26 @@ module Saviour
         )
       end.compact
 
-      pool = Concurrent::FixedThreadPool.new(Saviour::Config.concurrent_workers)
+      pool = Concurrent::Throttle.new Saviour::Config.concurrent_workers
 
       futures = uploaders.map { |uploader|
-        Concurrent::Future.execute(executor: pool) {
-          if defined?(Rails) && Rails::VERSION::MAJOR < 6
-            Rails.application.executor.wrap { uploader.upload }
+        pool.future(uploader) { |given_uploader|
+          if SHOULD_USE_INTERLOCK
+            Rails.application.executor.wrap { given_uploader.upload }
           else
-            uploader.upload
+            given_uploader.upload
           end
         }
       }
 
-      work = -> {
-        futures.map do |x|
-          x.value.tap do
-            raise(x.reason) if x.rejected?
-          end
-        end.compact
-      }
+      work = -> { futures.map(&:value!).compact }
 
-      result = if defined?(Rails) && Rails::VERSION::MAJOR < 6
+      result = if SHOULD_USE_INTERLOCK
                  ActiveSupport::Dependencies.interlock.permit_concurrent_loads(&work)
                else
                  work.call
                end
 
-      pool.shutdown
-      pool.wait_for_termination
-
       attrs = result.to_h
 
       uploaders.map(&:uploader).select { |x| x.class.after_upload_hooks.any? }.each do |uploader|

data/lib/saviour/version.rb

@@ -1,3 +1,3 @@
 module Saviour
-  VERSION = "0.6.6"
+  VERSION = "0.6.7"
 end

data/saviour.gemspec

@@ -18,6 +18,7 @@ Gem::Specification.new do |spec|
   spec.add_dependency "activerecord", ">= 5.1"
   spec.add_dependency "activesupport", ">= 5.1"
   spec.add_dependency "concurrent-ruby", ">= 1.0.5"
+  spec.add_dependency "concurrent-ruby-edge", ">= 0.6.0"
 
   spec.add_development_dependency "bundler"
   spec.add_development_dependency "rspec"

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: saviour
 version: !ruby/object:Gem::Version
-  version: 0.6.6
+  version: 0.6.7
 platform: ruby
 authors:
 - Roger Campos
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2019-10-16 00:00:00.000000000 Z
+date: 2020-06-05 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activerecord
@@ -52,6 +52,20 @@ dependencies:
     - - ">="
       - !ruby/object:Gem::Version
         version: 1.0.5
+- !ruby/object:Gem::Dependency
+  name: concurrent-ruby-edge
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: 0.6.0
+  type: :runtime
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: 0.6.0
 - !ruby/object:Gem::Dependency
   name: bundler
   requirement: !ruby/object:Gem::Requirement
@@ -241,8 +255,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubyforge_project: 
-rubygems_version: 2.7.6
+rubygems_version: 3.0.3
 signing_key: 
 specification_version: 4
 summary: File storage handler following active record model lifecycle