protobuf-nats 0.9.0.pre1 → 0.9.0.pre2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: b965b64f467078a814cd456821efec903fcff030
-  data.tar.gz: 83f040bbc6b8d48d5421fb9f31c44278b7f5ef63
+  metadata.gz: b7b7a2ede49bfda45aadc50f2cc978e9ac6f4780
+  data.tar.gz: a6a6253219abc6417bcf87079da8d61694dfe941
 SHA512:
-  metadata.gz: 2fce53579733468be93dece666e78ec435280fdb143d9155ea8302c43f70a4f59efcb5af8668eb5c2af85d1d415631fd5bcb2300bef8def906a3afc9ff58a7cd
-  data.tar.gz: 23bf375eeae757647bedd11c19f2b02cf70d9d4b841820003856e87c8148963e4e4ca5b446d17050beefd966c1ab109315b33f9cc3f3434a74ac336b0900e910
+  metadata.gz: 3a6d7bc54084e79bf44ab56e1847823b37523eca4be6587509cf2a22682bdffd86ea25a574acac65a2e7e0dd92e04a1a173e0230a0b3ca02da1bad02db107400
+  data.tar.gz: a25d3136a2c8767f13fc163f298f905d69f19970db022ccdee4ee8baffaea402a54448770a822fa5b58218936f6e3e84406a8fc316aa0fc6360e17932be9c3eb

data/.gitignore

@@ -1,3 +1,4 @@
+.ruby-*
 /.bundle/
 /.yardoc
 /Gemfile.lock

data/README.md

@@ -47,6 +47,8 @@ used to allow JVM based servers to warm-up slowly to prevent jolts in runtime pe
 
 `PB_NATS_CLIENT_RECONNECT_DELAY` - If we detect a reconnect delay, we will wait this many seconds (default: the ACK timeout).
 
+`PB_NATS_CLIENT_SUBSCRIPTION_POOL_SIZE` - If subscription pooling is desired for the request/response cycle then the pool size maximum should be set; the pool is lazy and therefore will only start new subscriptions as necessary (default: 0)
+
 `PROTOBUF_NATS_CONFIG_PATH` - Custom path to the config yaml (default: "config/protobuf_nats.yml").
 
 ### YAML Config

data/lib/protobuf/nats/client.rb

@@ -58,7 +58,7 @@ module Protobuf
             return_value = yield sub_inbox
           end
         else
-         return_value = yield new_subscription_inbox
+          return_value = yield new_subscription_inbox
         end
 
         return_value
@@ -128,7 +128,7 @@ module Protobuf
       def send_request
         if use_subscription_pooling?
           available = self.class.subscription_pool.instance_variable_get("@available")
-          ::ActiveSupport::Notifications.instrument "client.pool_availble_size.protobuf-nats", available.length
+          ::ActiveSupport::Notifications.instrument "client.subscription_pool_available_size.protobuf-nats", available.length
         end
 
         ::ActiveSupport::Notifications.instrument "client.request_duration.protobuf-nats" do
@@ -203,6 +203,12 @@ module Protobuf
           with_subscription do |sub_inbox|
             begin
               completed_request = false
+
+              if !sub_inbox.subscription.is_valid # replace the subscription if is has been pooled but is no longer valid (maybe a reconnect)
+                nats.unsubscribe(sub_inbox.subscription)
+                sub_inbox.swap(new_subscription_inbox) # this line replaces the sub_inbox in the connection pool if necessary
+              end
+
               nats.publish(subject, data, sub_inbox.inbox)
 
               # Wait for reply

data/lib/protobuf/nats/version.rb

@@ -1,5 +1,5 @@
 module Protobuf
   module Nats
-    VERSION = "0.9.0.pre1"
+    VERSION = "0.9.0.pre2"
   end
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: protobuf-nats
 version: !ruby/object:Gem::Version
-  version: 0.9.0.pre1
+  version: 0.9.0.pre2
 platform: ruby
 authors:
 - Brandon Dewitt
 autorequire: 
 bindir: exe
 cert_chain: []
-date: 2017-11-05 00:00:00.000000000 Z
+date: 2017-11-06 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activesupport
@@ -199,7 +199,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: 1.3.1
 requirements: []
 rubyforge_project: 
-rubygems_version: 2.5.1
+rubygems_version: 2.6.13
 signing_key: 
 specification_version: 4
 summary: ruby-protobuf client/server for nats