redis 3.3.5 → 4.1.4

data/lib/redis/errors.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 class Redis
   # Base error for all redis-rb errors.
   class BaseError < RuntimeError
@@ -37,4 +38,50 @@ class Redis
   # Raised when the connection was inherited by a child process.
   class InheritedError < BaseConnectionError
   end
+
+  # Raised when client options are invalid.
+  class InvalidClientOptionError < BaseError
+  end
+
+  class Cluster
+    # Raised when client connected to redis as cluster mode
+    # and some cluster subcommands were called.
+    class OrchestrationCommandNotSupported < BaseError
+      def initialize(command, subcommand = '')
+        str = [command, subcommand].map(&:to_s).reject(&:empty?).join(' ').upcase
+        msg = "#{str} command should be used with care "\
+              'only by applications orchestrating Redis Cluster, like redis-trib, '\
+              'and the command if used out of the right context can leave the cluster '\
+              'in a wrong state or cause data loss.'
+        super(msg)
+      end
+    end
+
+    # Raised when error occurs on any node of cluster.
+    class CommandErrorCollection < BaseError
+      attr_reader :errors
+
+      # @param errors [Hash{String => Redis::CommandError}]
+      # @param error_message [String]
+      def initialize(errors, error_message = 'Command errors were replied on any node')
+        @errors = errors
+        super(error_message)
+      end
+    end
+
+    # Raised when cluster client can't select node.
+    class AmbiguousNodeError < BaseError
+      def initialize(command)
+        super("Cluster client doesn't know which node the #{command} command should be sent to.")
+      end
+    end
+
+    # Raised when commands in pipelining include cross slot keys.
+    class CrossSlotPipeliningError < BaseError
+      def initialize(keys)
+        super("Cluster client couldn't send pipelining to single node. "\
+              "The commands include cross slot keys. #{keys}")
+      end
+    end
+  end
 end

data/lib/redis/hash_ring.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 require 'zlib'
 
 class Redis
@@ -25,7 +26,6 @@ class Redis
       @nodes << node
       @replicas.times do |i|
         key = Zlib.crc32("#{node.id}:#{i}")
-        raise "Node ID collision" if @ring.has_key?(key)
         @ring[key] = node
         @sorted_keys << key
       end
@@ -61,72 +61,29 @@ class Redis
       end
     end
 
-    class << self
-
-      # gem install RubyInline to use this code
-      # Native extension to perform the binary search within the hashring.
-      # There's a pure ruby version below so this is purely optional
-      # for performance.  In testing 20k gets and sets, the native
-      # binary search shaved about 12% off the runtime (9sec -> 8sec).
-      begin
-        require 'inline'
-        inline do |builder|
-          builder.c <<-EOM
-          int binary_search(VALUE ary, unsigned int r) {
-              int upper = RARRAY_LEN(ary) - 1;
-              int lower = 0;
-              int idx = 0;
-
-              while (lower <= upper) {
-                  idx = (lower + upper) / 2;
-
-                  VALUE continuumValue = RARRAY_PTR(ary)[idx];
-                  unsigned int l = NUM2UINT(continuumValue);
-                  if (l == r) {
-                      return idx;
-                  }
-                  else if (l > r) {
-                      upper = idx - 1;
-                  }
-                  else {
-                      lower = idx + 1;
-                  }
-              }
-              if (upper < 0) {
-                upper = RARRAY_LEN(ary) - 1;
-              }
-              return upper;
-          }
-          EOM
-        end
-      rescue Exception
-        # Find the closest index in HashRing with value <= the given value
-        def binary_search(ary, value, &block)
-          upper = ary.size - 1
-          lower = 0
-          idx = 0
-
-          while(lower <= upper) do
-            idx = (lower + upper) / 2
-            comp = ary[idx] <=> value
-
-            if comp == 0
-              return idx
-            elsif comp > 0
-              upper = idx - 1
-            else
-              lower = idx + 1
-            end
-          end
-
-          if upper < 0
-            upper = ary.size - 1
-          end
-          return upper
+    # Find the closest index in HashRing with value <= the given value
+    def self.binary_search(ary, value, &block)
+      upper = ary.size - 1
+      lower = 0
+      idx = 0
+
+      while(lower <= upper) do
+        idx = (lower + upper) / 2
+        comp = ary[idx] <=> value
+
+        if comp == 0
+          return idx
+        elsif comp > 0
+          upper = idx - 1
+        else
+          lower = idx + 1
         end
+      end
 
+      if upper < 0
+        upper = ary.size - 1
       end
+      return upper
     end
-
   end
 end

data/lib/redis/pipeline.rb

@@ -1,21 +1,22 @@
+# frozen_string_literal: true
 class Redis
-  unless defined?(::BasicObject)
-    class BasicObject
-      instance_methods.each { |meth| undef_method(meth) unless meth =~ /\A(__|instance_eval)/ }
-    end
-  end
-
   class Pipeline
     attr_accessor :db
+    attr_reader :client
 
     attr :futures
 
-    def initialize
+    def initialize(client)
+      @client = client.is_a?(Pipeline) ? client.client : client
       @with_reconnect = true
       @shutdown = false
       @futures = []
     end
 
+    def timeout
+      client.timeout
+    end
+
     def with_reconnect?
       @with_reconnect
     end
@@ -28,15 +29,23 @@ class Redis
       @shutdown
     end
 
-    def call(command, &block)
+    def empty?
+      @futures.empty?
+    end
+
+    def call(command, timeout: nil, &block)
       # A pipeline that contains a shutdown should not raise ECONNRESET when
       # the connection is gone.
       @shutdown = true if command.first == :shutdown
-      future = Future.new(command, block)
+      future = Future.new(command, block, timeout)
       @futures << future
       future
     end
 
+    def call_with_timeout(command, timeout, &block)
+      call(command, timeout: timeout, &block)
+    end
+
     def call_pipeline(pipeline)
       @shutdown = true if pipeline.shutdown?
       @futures.concat(pipeline.futures)
@@ -45,7 +54,11 @@ class Redis
     end
 
     def commands
-      @futures.map { |f| f._command }
+      @futures.map(&:_command)
+    end
+
+    def timeouts
+      @futures.map(&:timeout)
     end
 
     def with_reconnect(val=true)
@@ -91,8 +104,20 @@ class Redis
         end
       end
 
+      def timeouts
+        if empty?
+          []
+        else
+          [nil, *super, nil]
+        end
+      end
+
       def commands
-        [[:multi]] + super + [[:exec]]
+        if empty?
+          []
+        else
+          [[:multi]] + super + [[:exec]]
+        end
       end
     end
   end
@@ -106,12 +131,29 @@ class Redis
   class Future < BasicObject
     FutureNotReady = ::Redis::FutureNotReady.new
 
-    def initialize(command, transformation)
+    attr_reader :timeout
+
+    def initialize(command, transformation, timeout)
       @command = command
       @transformation = transformation
+      @timeout = timeout
       @object = FutureNotReady
     end
 
+    def ==(_other)
+      message = +"The methods == and != are deprecated for Redis::Future and will be removed in 4.2.0"
+      message << " - You probably meant to call .value == or .value !="
+      message << " (#{::Kernel.caller(1, 1).first})\n"
+
+      if defined?(::Warning)
+        ::Warning.warn(message)
+      else
+        $stderr.puts(message)
+      end
+
+      super
+    end
+
     def inspect
       "<Redis::Future #{@command.inspect}>"
     end

data/lib/redis/subscribe.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 class Redis
   class SubscribedClient
     def initialize(client)

data/lib/redis/version.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 class Redis
-  VERSION = "3.3.5"
+  VERSION = '4.1.4'
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: redis
 version: !ruby/object:Gem::Version
-  version: 3.3.5
+  version: 4.1.4
 platform: ruby
 authors:
 - Ezra Zygmuntowicz
@@ -16,74 +16,74 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2017-09-28 00:00:00.000000000 Z
+date: 2020-05-01 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
-  name: rake
+  name: mocha
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - "<"
+    - - ">="
       - !ruby/object:Gem::Version
-        version: 11.0.0
+        version: '0'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - "<"
+    - - ">="
       - !ruby/object:Gem::Version
-        version: 11.0.0
+        version: '0'
 - !ruby/object:Gem::Dependency
-  name: test-unit
+  name: hiredis
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - '='
+    - - ">="
       - !ruby/object:Gem::Version
-        version: 3.1.5
+        version: '0'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - '='
+    - - ">="
       - !ruby/object:Gem::Version
-        version: 3.1.5
+        version: '0'
+- !ruby/object:Gem::Dependency
+  name: em-synchrony
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
 description: |2
       A Ruby client that tries to match Redis' API one-to-one, while still
-      providing an idiomatic interface. It features thread-safety,
-      client-side sharding, pipelining, and an obsession for performance.
+      providing an idiomatic interface.
 email:
 - redis-db@googlegroups.com
 executables: []
 extensions: []
 extra_rdoc_files: []
 files:
-- ".gitignore"
-- ".travis.yml"
-- ".travis/Gemfile"
-- ".yardopts"
 - CHANGELOG.md
-- Gemfile
 - LICENSE
 - README.md
-- Rakefile
-- benchmarking/logging.rb
-- benchmarking/pipeline.rb
-- benchmarking/speed.rb
-- benchmarking/suite.rb
-- benchmarking/worker.rb
-- examples/basic.rb
-- examples/consistency.rb
-- examples/dist_redis.rb
-- examples/incr-decr.rb
-- examples/list.rb
-- examples/pubsub.rb
-- examples/sentinel.rb
-- examples/sentinel/sentinel.conf
-- examples/sentinel/start
-- examples/sets.rb
-- examples/unicorn/config.ru
-- examples/unicorn/unicorn.rb
 - lib/redis.rb
 - lib/redis/client.rb
+- lib/redis/cluster.rb
+- lib/redis/cluster/command.rb
+- lib/redis/cluster/command_loader.rb
+- lib/redis/cluster/key_slot_converter.rb
+- lib/redis/cluster/node.rb
+- lib/redis/cluster/node_key.rb
+- lib/redis/cluster/node_loader.rb
+- lib/redis/cluster/option.rb
+- lib/redis/cluster/slot.rb
+- lib/redis/cluster/slot_loader.rb
 - lib/redis/connection.rb
 - lib/redis/connection/command_helper.rb
 - lib/redis/connection/hiredis.rb
@@ -96,85 +96,6 @@ files:
 - lib/redis/pipeline.rb
 - lib/redis/subscribe.rb
 - lib/redis/version.rb
-- redis.gemspec
-- test/bitpos_test.rb
-- test/blocking_commands_test.rb
-- test/client_test.rb
-- test/command_map_test.rb
-- test/commands_on_hashes_test.rb
-- test/commands_on_hyper_log_log_test.rb
-- test/commands_on_lists_test.rb
-- test/commands_on_sets_test.rb
-- test/commands_on_sorted_sets_test.rb
-- test/commands_on_strings_test.rb
-- test/commands_on_value_types_test.rb
-- test/connection_handling_test.rb
-- test/connection_test.rb
-- test/db/.gitkeep
-- test/distributed_blocking_commands_test.rb
-- test/distributed_commands_on_hashes_test.rb
-- test/distributed_commands_on_hyper_log_log_test.rb
-- test/distributed_commands_on_lists_test.rb
-- test/distributed_commands_on_sets_test.rb
-- test/distributed_commands_on_sorted_sets_test.rb
-- test/distributed_commands_on_strings_test.rb
-- test/distributed_commands_on_value_types_test.rb
-- test/distributed_commands_requiring_clustering_test.rb
-- test/distributed_connection_handling_test.rb
-- test/distributed_internals_test.rb
-- test/distributed_key_tags_test.rb
-- test/distributed_persistence_control_commands_test.rb
-- test/distributed_publish_subscribe_test.rb
-- test/distributed_remote_server_control_commands_test.rb
-- test/distributed_scripting_test.rb
-- test/distributed_sorting_test.rb
-- test/distributed_test.rb
-- test/distributed_transactions_test.rb
-- test/encoding_test.rb
-- test/error_replies_test.rb
-- test/fork_safety_test.rb
-- test/helper.rb
-- test/helper_test.rb
-- test/internals_test.rb
-- test/lint/blocking_commands.rb
-- test/lint/hashes.rb
-- test/lint/hyper_log_log.rb
-- test/lint/lists.rb
-- test/lint/sets.rb
-- test/lint/sorted_sets.rb
-- test/lint/strings.rb
-- test/lint/value_types.rb
-- test/persistence_control_commands_test.rb
-- test/pipelining_commands_test.rb
-- test/publish_subscribe_test.rb
-- test/remote_server_control_commands_test.rb
-- test/scanning_test.rb
-- test/scripting_test.rb
-- test/sentinel_command_test.rb
-- test/sentinel_test.rb
-- test/sorting_test.rb
-- test/ssl_test.rb
-- test/support/connection/hiredis.rb
-- test/support/connection/ruby.rb
-- test/support/connection/synchrony.rb
-- test/support/redis_mock.rb
-- test/support/ssl/gen_certs.sh
-- test/support/ssl/trusted-ca.crt
-- test/support/ssl/trusted-ca.key
-- test/support/ssl/trusted-cert.crt
-- test/support/ssl/trusted-cert.key
-- test/support/ssl/untrusted-ca.crt
-- test/support/ssl/untrusted-ca.key
-- test/support/ssl/untrusted-cert.crt
-- test/support/ssl/untrusted-cert.key
-- test/support/wire/synchrony.rb
-- test/support/wire/thread.rb
-- test/synchrony_driver.rb
-- test/test.conf.erb
-- test/thread_safety_test.rb
-- test/transactions_test.rb
-- test/unknown_commands_test.rb
-- test/url_param_test.rb
 homepage: https://github.com/redis/redis-rb
 licenses:
 - MIT
@@ -187,94 +108,15 @@ required_ruby_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
-      version: '0'
+      version: 2.3.0
 required_rubygems_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubyforge_project: 
-rubygems_version: 2.5.1
+rubygems_version: 3.0.4
 signing_key: 
 specification_version: 4
 summary: A Ruby client library for Redis
-test_files:
-- test/bitpos_test.rb
-- test/blocking_commands_test.rb
-- test/client_test.rb
-- test/command_map_test.rb
-- test/commands_on_hashes_test.rb
-- test/commands_on_hyper_log_log_test.rb
-- test/commands_on_lists_test.rb
-- test/commands_on_sets_test.rb
-- test/commands_on_sorted_sets_test.rb
-- test/commands_on_strings_test.rb
-- test/commands_on_value_types_test.rb
-- test/connection_handling_test.rb
-- test/connection_test.rb
-- test/db/.gitkeep
-- test/distributed_blocking_commands_test.rb
-- test/distributed_commands_on_hashes_test.rb
-- test/distributed_commands_on_hyper_log_log_test.rb
-- test/distributed_commands_on_lists_test.rb
-- test/distributed_commands_on_sets_test.rb
-- test/distributed_commands_on_sorted_sets_test.rb
-- test/distributed_commands_on_strings_test.rb
-- test/distributed_commands_on_value_types_test.rb
-- test/distributed_commands_requiring_clustering_test.rb
-- test/distributed_connection_handling_test.rb
-- test/distributed_internals_test.rb
-- test/distributed_key_tags_test.rb
-- test/distributed_persistence_control_commands_test.rb
-- test/distributed_publish_subscribe_test.rb
-- test/distributed_remote_server_control_commands_test.rb
-- test/distributed_scripting_test.rb
-- test/distributed_sorting_test.rb
-- test/distributed_test.rb
-- test/distributed_transactions_test.rb
-- test/encoding_test.rb
-- test/error_replies_test.rb
-- test/fork_safety_test.rb
-- test/helper.rb
-- test/helper_test.rb
-- test/internals_test.rb
-- test/lint/blocking_commands.rb
-- test/lint/hashes.rb
-- test/lint/hyper_log_log.rb
-- test/lint/lists.rb
-- test/lint/sets.rb
-- test/lint/sorted_sets.rb
-- test/lint/strings.rb
-- test/lint/value_types.rb
-- test/persistence_control_commands_test.rb
-- test/pipelining_commands_test.rb
-- test/publish_subscribe_test.rb
-- test/remote_server_control_commands_test.rb
-- test/scanning_test.rb
-- test/scripting_test.rb
-- test/sentinel_command_test.rb
-- test/sentinel_test.rb
-- test/sorting_test.rb
-- test/ssl_test.rb
-- test/support/connection/hiredis.rb
-- test/support/connection/ruby.rb
-- test/support/connection/synchrony.rb
-- test/support/redis_mock.rb
-- test/support/ssl/gen_certs.sh
-- test/support/ssl/trusted-ca.crt
-- test/support/ssl/trusted-ca.key
-- test/support/ssl/trusted-cert.crt
-- test/support/ssl/trusted-cert.key
-- test/support/ssl/untrusted-ca.crt
-- test/support/ssl/untrusted-ca.key
-- test/support/ssl/untrusted-cert.crt
-- test/support/ssl/untrusted-cert.key
-- test/support/wire/synchrony.rb
-- test/support/wire/thread.rb
-- test/synchrony_driver.rb
-- test/test.conf.erb
-- test/thread_safety_test.rb
-- test/transactions_test.rb
-- test/unknown_commands_test.rb
-- test/url_param_test.rb
+test_files: []