protobuf-nats 0.9.0 → 0.10.3

checksums.yaml

@@ -1,7 +1,7 @@
 ---
-SHA1:
-  metadata.gz: 7c49c28f4b4cb3b40b36deb9f75b3bd3946cc5b0
-  data.tar.gz: 95e4b3d9a014fef6f693b344f21d4deaef7ce9ad
+SHA256:
+  metadata.gz: 167e2b896360bdf341fbdc1812de23e69cf28c2bf25057180fa68507176d8c55
+  data.tar.gz: 6ee287bffca6e3b5efeba47f76a43be764f38bee331a8d3164d3d1592dd132b4
 SHA512:
-  metadata.gz: 3e0259df6a6b6612720a21764aaf61ea7462cb42ec424c2690892ff40ff532d6f8bcff7d838294728df50ec54255245b193dcf491358d8b290c13ebf5b4e5b69
-  data.tar.gz: a75492a597508f386c4bc30714b20de27da114f4578e9f928e722e66d4f13c3f775294f684d7f3eb169445af2663775c82ffd4a9d65ec61537b9b52520d36f58
+  metadata.gz: 516f3f999e4c332fd3b211455d57514f2c01f13ef8d0cac8cd2d846c5139471d2c0f4cf63d839d3c3ecd3383b9782f0ac554c43923a8e611c8a1dd9b498419b2
+  data.tar.gz: f50854739d873a49f920f2cdabd229315722ebc42c833598b4f6a1cdcb5a6d4ca7040eb2a710487b7741270c28cb3314388492b488ab931677409d785bc6e7c5

data/.travis.yml

@@ -1,8 +1,16 @@
 sudo: false
 language: ruby
+jdk:
+ - openjdk8
 rvm:
   - 2.3.0
+  - 2.7.0
   - jruby-9.1.7.0
+  - jruby-9.2.13.0
 before_install:
-  - gem install bundler -v 1.14.3
+  # Install and start gnatsd
+  - ./scripts/install_gnatsd.sh
+  - $HOME/nats-server/nats-server &
+  # Install deps for project
+  - gem install bundler
   - gem update --system

data/README.md

@@ -154,6 +154,8 @@ After checking out the repo, run `bin/setup` to install dependencies. Then, run
 
 To install this gem onto your local machine, run `bundle exec rake install`. To release a new version, update the version number in `version.rb`, and then run `bundle exec rake release`, which will create a git tag for the version, push git commits and tags, and push the `.gem` file to [rubygems.org](https://rubygems.org).
 
+The java-nats client is temporarily forked to support jruby > 9.2.10.0. The living branch for that is here: https://github.com/film42/java-nats/tree/jruby-compat. This will be removed when we upgrade to the new nats.java client.
+
 ## Contributing
 
 Bug reports and pull requests are welcome on GitHub at https://github.com/abrandoned/protobuf-nats.

data/lib/protobuf/nats/client.rb

@@ -151,12 +151,12 @@ module Protobuf
           when :ack_timeout
             ::ActiveSupport::Notifications.instrument "client.request_timeout.protobuf-nats"
             next if (retries -= 1) > 0
-            raise ::Protobuf::Nats::Errors::RequestTimeout
+            raise ::Protobuf::Nats::Errors::RequestTimeout, formatted_service_and_method_name
           when :nack
             ::ActiveSupport::Notifications.instrument "client.request_nack.protobuf-nats"
             interval = nack_backoff_intervals[nack_retry]
             nack_retry += 1
-            raise ::Protobuf::Nats::Errors::RequestTimeout if interval.nil?
+            raise ::Protobuf::Nats::Errors::RequestTimeout, formatted_service_and_method_name if interval.nil?
             sleep((interval + nack_backoff_splay)/1000.0)
             next
           end
@@ -186,6 +186,12 @@ module Protobuf
         end
       end
 
+      def formatted_service_and_method_name
+        klass = @options[:service]
+        method_name = @options[:method]
+        "#{klass}##{method_name}"
+      end
+
       # The Java nats client offers better message queueing so we're going to use
       # that over locking ourselves. This split in code isn't great, but we can
       # refactor this later.
@@ -231,7 +237,7 @@ module Protobuf
                          else return :ack_timeout
                          end
 
-              fail(::Protobuf::Nats::Errors::ResponseTimeout, subject) unless response
+              fail(::Protobuf::Nats::Errors::ResponseTimeout, formatted_service_and_method_name) unless response
 
               completed_request = true
               response
@@ -287,7 +293,7 @@ module Protobuf
                      else return :ack_timeout
                      end
 
-          fail(::Protobuf::Nats::Errors::ResponseTimeout, subject) unless response
+          fail(::Protobuf::Nats::Errors::ResponseTimeout, formatted_service_and_method_name) unless response
 
           response
         ensure

data/lib/protobuf/nats/errors.rb

@@ -1,13 +1,13 @@
 module Protobuf
   module Nats
     module Errors
-      class Base < ::StandardError
+      class ClientError < ::StandardError
       end
 
-      class RequestTimeout < Base
+      class RequestTimeout < ClientError
       end
 
-      class ResponseTimeout < Base
+      class ResponseTimeout < ClientError
       end
 
       class MriIOException < ::StandardError

data/lib/protobuf/nats/jnats.rb

@@ -5,11 +5,6 @@ require ::File.join(ext_base, "jars/slf4j-simple-1.7.25.jar")
 require ::File.join(ext_base, "jars/gson-2.6.2.jar")
 require ::File.join(ext_base, "jars/jnats-1.1-SNAPSHOT.jar")
 
-# Set field accessors so we can access the member variables directly.
-class Java::IoNatsClient::SubscriptionImpl
-  field_accessor :pMsgs, :pBytes, :delivered
-end
-
 module Protobuf
   module Nats
     class JNats
@@ -190,9 +185,9 @@ module Protobuf
               # We have to update the subscription stats so we're not considered a slow consumer.
               begin
                 sub.lock
-                sub.pMsgs -= 1
-                sub.pBytes -= message.getData.length if message.getData
-                sub.delivered += 1 unless sub.isClosed
+                sub.incrPMsgs(-1)
+                sub.incrPBytes(-message.getData.length) if message.getData
+                sub.incrDelivered(1) unless sub.isClosed
               ensure
                 sub.unlock
               end

data/lib/protobuf/nats/server.rb

@@ -12,6 +12,8 @@ module Protobuf
 
       attr_reader :nats, :thread_pool, :subscriptions
 
+      MILLISECOND = 1000
+
       def initialize(options)
         @options = options
         @processing_requests = true
@@ -52,7 +54,7 @@ module Protobuf
             # Instrument the thread pool time-to-execute duration.
             processed_at = ::Time.now
             ::ActiveSupport::Notifications.instrument("server.thread_pool_execution_delay.protobuf-nats",
-                                                      (processed_at - enqueued_at))
+                                                      (processed_at - enqueued_at) * MILLISECOND)
 
             # Process request.
             response_data = handle_request(request_data, 'server' => @server)
@@ -64,7 +66,7 @@ module Protobuf
             # Instrument the request duration.
             completed_at = ::Time.now
             ::ActiveSupport::Notifications.instrument("server.request_duration.protobuf-nats",
-                                                      (completed_at - enqueued_at))
+                                                      (completed_at - enqueued_at) * MILLISECOND)
           end
         end
 

data/lib/protobuf/nats/version.rb

@@ -1,5 +1,5 @@
 module Protobuf
   module Nats
-    VERSION = "0.9.0"
+    VERSION = "0.10.3"
   end
 end

data/protobuf-nats.gemspec

@@ -33,9 +33,9 @@ Gem::Specification.new do |spec|
   spec.add_runtime_dependency "activesupport", ">= 3.2"
   spec.add_runtime_dependency "connection_pool"
   spec.add_runtime_dependency "protobuf", "~> 3.7", ">= 3.7.2"
-  spec.add_runtime_dependency "nats-pure"
+  spec.add_runtime_dependency "nats-pure", "~> 0.3", "< 0.4"
 
-  spec.add_development_dependency "bundler", "~> 1.14"
+  spec.add_development_dependency "bundler"
   spec.add_development_dependency "rake", "~> 10.0"
   spec.add_development_dependency "rspec"
   spec.add_development_dependency "benchmark-ips"

data/scripts/install_gnatsd.sh

@@ -0,0 +1,20 @@
+#!/bin/bash
+
+set -e
+
+export DEFAULT_NATS_SERVER_VERSION=v2.0.0
+
+export NATS_SERVER_VERSION="${NATS_SERVER_VERSION:=$DEFAULT_NATS_SERVER_VERSION}"
+
+# check to see if nats-server folder is empty
+if [ ! "$(ls -A $HOME/nats-server)" ]; then
+    (
+	mkdir -p $HOME/nats-server
+	cd $HOME/nats-server
+	wget https://github.com/nats-io/nats-server/releases/download/$NATS_SERVER_VERSION/nats-server-$NATS_SERVER_VERSION-linux-amd64.zip -O nats-server.zip
+	unzip nats-server.zip
+	cp nats-server-$NATS_SERVER_VERSION-linux-amd64/nats-server $HOME/nats-server/nats-server
+    )
+else
+  echo 'Using cached directory.';
+fi

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: protobuf-nats
 version: !ruby/object:Gem::Version
-  version: 0.9.0
+  version: 0.10.3
 platform: ruby
 authors:
 - Brandon Dewitt
-autorequire: 
+autorequire:
 bindir: exe
 cert_chain: []
-date: 2017-11-08 00:00:00.000000000 Z
+date: 2020-08-25 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activesupport
@@ -62,30 +62,36 @@ dependencies:
   name: nats-pure
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - ">="
+    - - "~>"
       - !ruby/object:Gem::Version
-        version: '0'
+        version: '0.3'
+    - - "<"
+      - !ruby/object:Gem::Version
+        version: '0.4'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - ">="
+    - - "~>"
       - !ruby/object:Gem::Version
-        version: '0'
+        version: '0.3'
+    - - "<"
+      - !ruby/object:Gem::Version
+        version: '0.4'
 - !ruby/object:Gem::Dependency
   name: bundler
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ">="
       - !ruby/object:Gem::Version
-        version: '1.14'
+        version: '0'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ">="
       - !ruby/object:Gem::Version
-        version: '1.14'
+        version: '0'
 - !ruby/object:Gem::Dependency
   name: rake
   requirement: !ruby/object:Gem::Requirement
@@ -178,12 +184,13 @@ files:
 - lib/protobuf/nats/thread_pool.rb
 - lib/protobuf/nats/version.rb
 - protobuf-nats.gemspec
-homepage: 
+- scripts/install_gnatsd.sh
+homepage:
 licenses:
 - MIT
 metadata:
   allowed_push_host: https://rubygems.org
-post_install_message: 
+post_install_message:
 rdoc_options: []
 require_paths:
 - lib
@@ -198,9 +205,8 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubyforge_project: 
-rubygems_version: 2.6.13
-signing_key: 
+rubygems_version: 3.1.2
+signing_key:
 specification_version: 4
 summary: ruby-protobuf client/server for nats
 test_files: []