promiscuous 1.0.0.beta4 → 1.0.0.beta5

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: a2be6a0c5f3e918b401df4b70498f2120e2493e1
-  data.tar.gz: 14a8ff258fd6cb14d59d7ad8efb874a37bb5424e
+  metadata.gz: 80819caeaff72e0d2634206a082cd8ba0686d5f3
+  data.tar.gz: ee744941f13a6dada05b5781f5e5600856ee00d4
 SHA512:
-  metadata.gz: d98fd116d3dd2e123a703cca6feb25be23a75ad139df43f0c0e7a1a2091ceb84d782c46ae455f06385f4136d69533ea01243389ab42c5719a1970b68b59e28a4
-  data.tar.gz: de9801711b20a656c81fc53fcf74c67015e8bd9e9b25c8af1c20a3d0045e02a5008c0b2f3d427ec84e5bc369aa15434ebbe5d11d45a46156043cf2cb3ccdeec8
+  metadata.gz: efbd90b25ff49949dd54ac5c2ee06300e5e06ff0cbd9c875d0608dadff15e682062874ec3a7c20ab10589706d1de5e5235e5ec6119246fb3b718577305e54096
+  data.tar.gz: 0c55d26471375d3933f61bb24c8ee9490b6268db57e4d93d7ea98ecab12dfa2e9bfc3fd7fde099a9da9f43524936c5a91801960980834e139b8322e1d7fafa6a

data/lib/promiscuous/amqp/bunny.rb

@@ -89,8 +89,8 @@ class Promiscuous::AMQP::Bunny
       @callback_mapping[tag] = options[:on_confirm] if options[:on_confirm]
     end
   rescue Exception => e
+    Promiscuous.warn("[publish] Failure publishing to rabbit #{e}\n#{e.backtrace.join("\n")}")
     e = Promiscuous::Error::Publisher.new(e, :payload => options[:payload])
-    Promiscuous.warn "[publish] #{e}\n#{e.backtrace.join("\n")}"
     Promiscuous::Config.error_notifier.call(e)
   end
 

data/lib/promiscuous/publisher/operation/mongoid.rb

@@ -201,7 +201,7 @@ class Moped::PromiscuousQueryWrapper < Moped::Query
   end
 
   def remove_all
-    raise "Instead of doing a multi delete, delete each document separatly.\n" +
+    raise "Promiscuous: Instead of doing a multi delete, delete each document separatly.\n" +
           "Declare your has_many relationships with :dependent => :destroy instead of :delete"
   end
 end

data/lib/promiscuous/publisher/transport/batch.rb

@@ -46,7 +46,10 @@ class Promiscuous::Publisher::Transport::Batch
       end
     rescue Exception => e
       Promiscuous.warn("[publish] Failure publishing to rabbit #{e}\n#{e.backtrace.join("\n")}")
-      raise e if raise_error
+      e = Promiscuous::Error::Publisher.new(e, :payload => self.payload)
+      Promiscuous::Config.error_notifier.call(e)
+
+      raise e.inner if raise_error
     end
   end
 
@@ -90,7 +93,7 @@ class Promiscuous::Publisher::Transport::Batch
         else
           find_instance(attributes)
         end
-      end
+      end.compact
       self.new(dump['type'], instances)
     end
 

data/lib/promiscuous/publisher/transport/persistence/mongoid.rb

@@ -9,7 +9,7 @@ class Promiscuous::Publisher::Transport::Persistence::Mongoid
   end
 
   def delete(batch)
-    Storage.find(batch.id).destroy
+    Storage.collection.find(:_id => batch.id).remove
   end
 
   class Storage

data/lib/promiscuous/redis.rb

@@ -18,7 +18,7 @@ class Promiscuous::Redis
   end
 
   def self.disconnect
-    self.connection.quit
+    self.connection.try(:quit)
   end
 
   def self.lost_connection_exception(node, options={})

data/lib/promiscuous/subscriber/model/active_record.rb

@@ -31,5 +31,9 @@ module Promiscuous::Subscriber::Model::ActiveRecord
         promiscuous_root_class.find_by(key => id)
       end
     end
+
+    def __promiscuous_with_pooled_connection
+      ActiveRecord::Base.connection_pool.with_connection { yield }
+    end
   end
 end

data/lib/promiscuous/subscriber/model/base.rb

@@ -101,5 +101,9 @@ module Promiscuous::Subscriber::Model::Base
     def __promiscuous_fetch_new(id)
       new.tap { |m| m.__send__("#{subscribe_foreign_key}=", id) }
     end
+
+    def __promiscuous_with_pooled_connection
+      yield
+    end
   end
 end

data/lib/promiscuous/subscriber/worker/eventual_destroyer.rb

@@ -35,7 +35,7 @@ class Promiscuous::Subscriber::Worker::EventualDestroyer
     def perform
       model = class_name.constantize
       begin
-        model.__promiscuous_fetch_existing(instance_id).destroy
+        model.__promiscuous_with_pooled_connection { model.__promiscuous_fetch_existing(instance_id).destroy }
       rescue model.__promiscuous_missing_record_exception
       end
 

data/lib/promiscuous/version.rb

@@ -1,3 +1,3 @@
 module Promiscuous
-  VERSION = '1.0.0.beta4'
+  VERSION = '1.0.0.beta5'
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: promiscuous
 version: !ruby/object:Gem::Version
-  version: 1.0.0.beta4
+  version: 1.0.0.beta5
 platform: ruby
 authors:
 - Nicolas Viennot
@@ -9,104 +9,104 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2014-06-03 00:00:00.000000000 Z
+date: 2014-07-03 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activesupport
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - '>='
+    - - ">="
       - !ruby/object:Gem::Version
         version: '3'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - '>='
+    - - ">="
       - !ruby/object:Gem::Version
         version: '3'
 - !ruby/object:Gem::Dependency
   name: activemodel
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - '>='
+    - - ">="
       - !ruby/object:Gem::Version
         version: '3'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - '>='
+    - - ">="
       - !ruby/object:Gem::Version
         version: '3'
 - !ruby/object:Gem::Dependency
   name: bunny
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - '>='
+    - - ">="
       - !ruby/object:Gem::Version
         version: 0.10.7
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - '>='
+    - - ">="
       - !ruby/object:Gem::Version
         version: 0.10.7
 - !ruby/object:Gem::Dependency
   name: amq-protocol
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - '>='
+    - - ">="
       - !ruby/object:Gem::Version
         version: 1.8.0
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - '>='
+    - - ">="
       - !ruby/object:Gem::Version
         version: 1.8.0
 - !ruby/object:Gem::Dependency
   name: ruby-progressbar
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - ~>
+    - - "~>"
       - !ruby/object:Gem::Version
         version: 1.2.0
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - ~>
+    - - "~>"
       - !ruby/object:Gem::Version
         version: 1.2.0
 - !ruby/object:Gem::Dependency
   name: redis
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - ~>
+    - - "~>"
       - !ruby/object:Gem::Version
-        version: 3.0.2
+        version: '3.0'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - ~>
+    - - "~>"
       - !ruby/object:Gem::Version
-        version: 3.0.2
+        version: '3.0'
 - !ruby/object:Gem::Dependency
   name: algorithms
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - ~>
+    - - "~>"
       - !ruby/object:Gem::Version
         version: 0.6.1
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - ~>
+    - - "~>"
       - !ruby/object:Gem::Version
         version: 0.6.1
 - !ruby/object:Gem::Dependency
@@ -127,28 +127,28 @@ dependencies:
   name: multi_json
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - ~>
+    - - "~>"
       - !ruby/object:Gem::Version
         version: 1.8.0
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - ~>
+    - - "~>"
       - !ruby/object:Gem::Version
         version: 1.8.0
 - !ruby/object:Gem::Dependency
   name: robust-redis-lock
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - ~>
+    - - "~>"
       - !ruby/object:Gem::Version
         version: 0.2.2
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - ~>
+    - - "~>"
       - !ruby/object:Gem::Version
         version: 0.2.2
 description: Replicate models across applications
@@ -239,19 +239,18 @@ require_paths:
 - lib
 required_ruby_version: !ruby/object:Gem::Requirement
   requirements:
-  - - '>='
+  - - ">="
     - !ruby/object:Gem::Version
       version: '0'
 required_rubygems_version: !ruby/object:Gem::Requirement
   requirements:
-  - - '>'
+  - - ">"
     - !ruby/object:Gem::Version
       version: 1.3.1
 requirements: []
 rubyforge_project: 
-rubygems_version: 2.0.14
+rubygems_version: 2.1.11
 signing_key: 
 specification_version: 4
 summary: Replicate models across applications
 test_files: []
-has_rdoc: false