redis 1.0.5 → 1.0.6

data/lib/redis.rb

@@ -1,11 +1,15 @@
 require 'socket'
 
 class Redis
-  VERSION = "1.0.5"
+  VERSION = "1.0.6"
 
   def self.new(*attrs)
     Client.new(*attrs)
   end
+
+  def self.deprecate(message, trace = caller[0])
+    $stderr.puts "\n#{message} (in #{trace})"
+  end
 end
 
 begin

data/lib/redis/client.rb

@@ -63,10 +63,16 @@ class Redis
       "hset"      => BOOLEAN_PROCESSOR,
       "hexists"   => BOOLEAN_PROCESSOR,
       "info"      => lambda{|r|
-        info = {}
+        info = Hash.new do |hash, key|
+          if hash.include?(key.to_s)
+            Redis.deprecate "Redis#info will return a hash of string keys, not symbols", caller[2]
+            hash[key.to_s]
+          end
+        end
+
         r.each_line {|kv|
           k,v = kv.split(":",2).map{|x| x.chomp}
-          info[k.to_sym] = v
+          info[k] = v
         }
         info
       },
@@ -266,6 +272,7 @@ class Redis
     end
 
     def pipelined(&block)
+      Redis.deprecate("Calling pipelined commands on the yielded object will be deprecated in 2.0", caller[0])
       pipeline = Pipeline.new self
       yield pipeline
       pipeline.execute
@@ -496,8 +503,10 @@ class Redis
       @logger.send(level, str.to_s) if @logger
     end
 
-    def deprecated(old, new, trace = caller[0])
-      $stderr.puts "\nRedis: The method #{old} is deprecated. Use #{new} instead (in #{trace})"
+    def deprecated(old, new = nil, trace = caller[0])
+      message = "The method #{old} is deprecated and will be removed in 2.0"
+      message << " - use #{new} instead" if new
+      Redis.deprecate(message, trace)
     end
 
     def requires_timeout_reset?(command)

data/lib/redis/dist_redis.rb

@@ -1,115 +1,13 @@
-require 'redis/hash_ring'
+Redis.deprecate %q{"redis/dist_redis" is deprecated. Require "redis/distributed" and replace DistRedis for Redis::Distributed.}, caller[0]
 
-class Redis
-  class DistRedis
-    attr_reader :ring
-    def initialize(opts={})
-      hosts = []
-
-      db = opts[:db] || nil
-      timeout = opts[:timeout] || nil
-
-      raise "No hosts given" unless opts[:hosts]
-
-      opts[:hosts].each do |h|
-        host, port = h.split(':')
-        hosts << Client.new(:host => host, :port => port, :db => db, :timeout => timeout)
-      end
-
-      @ring = HashRing.new hosts
-    end
-
-    def node_for_key(key)
-      key = $1 if key =~ /\{(.*)?\}/
-      @ring.get_node(key)
-    end
-
-    def add_server(server)
-      server, port = server.split(':')
-      @ring.add_node Client.new(:host => server, :port => port)
-    end
-
-    def method_missing(sym, *args, &blk)
-      if redis = node_for_key(args.first.to_s)
-        redis.send sym, *args, &blk
-      else
-        super
-      end
-    end
-
-    def node_keys(glob)
-      @ring.nodes.map do |red|
-        red.keys(glob)
-      end
-    end
-
-    def keys(glob)
-      node_keys(glob).flatten
-    end
-
-    def save
-      on_each_node :save
-    end
-
-    def bgsave
-      on_each_node :bgsave
-    end
-
-    def quit
-      on_each_node :quit
-    end
-
-    def flush_all
-      on_each_node :flush_all
-    end
-    alias_method :flushall, :flush_all
+require "redis/hash_ring"
+require "redis/distributed"
 
-    def flush_db
-      on_each_node :flush_db
-    end
-    alias_method :flushdb, :flush_db
-
-    def delete_cloud!
-      @ring.nodes.each do |red|
-        red.keys("*").each do |key|
-          red.del key
-        end
-      end
-    end
-
-    def on_each_node(command, *args)
-      @ring.nodes.each do |red|
-        red.send(command, *args)
-      end
-    end
-
-    def mset()
-
-    end
-
-    def mget(*keyz)
-      results = {}
-      kbn = keys_by_node(keyz)
-      kbn.each do |node, node_keyz|
-        node.mapped_mget(*node_keyz).each do |k, v|
-          results[k] = v
-        end
-      end
-      keyz.flatten.map { |k| results[k] }
-    end
-
-    def keys_by_node(*keyz)
-      keyz.flatten.inject({}) do |kbn, k|
-        node = node_for_key(k)
-        next if kbn[node] && kbn[node].include?(k)
-        kbn[node] ||= []
-        kbn[node] << k
-        kbn
-      end
-    end
-
-    def type(key)
-      method_missing(:type, key)
+class Redis
+  class DistRedis < Redis::Distributed
+    def initialize(*args)
+      Redis.deprecate "DistRedis is deprecated in favor of Redis::Distributed.", caller[1]
+      super(*args)
     end
   end
 end

data/lib/redis/distributed.rb

@@ -0,0 +1,115 @@
+require 'redis/hash_ring'
+
+class Redis
+  class Distributed
+    attr_reader :ring
+    def initialize(opts={})
+      hosts = []
+
+      db = opts[:db] || nil
+      timeout = opts[:timeout] || nil
+
+      raise "No hosts given" unless opts[:hosts]
+
+      opts[:hosts].each do |h|
+        host, port = h.split(':')
+        hosts << Client.new(:host => host, :port => port, :db => db, :timeout => timeout)
+      end
+
+      @ring = HashRing.new hosts
+    end
+
+    def node_for_key(key)
+      key = $1 if key =~ /\{(.*)?\}/
+      @ring.get_node(key)
+    end
+
+    def add_server(server)
+      server, port = server.split(':')
+      @ring.add_node Client.new(:host => server, :port => port)
+    end
+
+    def method_missing(sym, *args, &blk)
+      if redis = node_for_key(args.first.to_s)
+        redis.send sym, *args, &blk
+      else
+        super
+      end
+    end
+
+    def node_keys(glob)
+      @ring.nodes.map do |red|
+        red.keys(glob)
+      end
+    end
+
+    def keys(glob)
+      node_keys(glob).flatten
+    end
+
+    def save
+      on_each_node :save
+    end
+
+    def bgsave
+      on_each_node :bgsave
+    end
+
+    def quit
+      on_each_node :quit
+    end
+
+    def flush_all
+      on_each_node :flush_all
+    end
+    alias_method :flushall, :flush_all
+
+    def flush_db
+      on_each_node :flush_db
+    end
+    alias_method :flushdb, :flush_db
+
+    def delete_cloud!
+      @ring.nodes.each do |red|
+        red.keys("*").each do |key|
+          red.del key
+        end
+      end
+    end
+
+    def on_each_node(command, *args)
+      @ring.nodes.each do |red|
+        red.send(command, *args)
+      end
+    end
+
+    def mset()
+
+    end
+
+    def mget(*keyz)
+      results = {}
+      kbn = keys_by_node(keyz)
+      kbn.each do |node, node_keyz|
+        node.mapped_mget(*node_keyz).each do |k, v|
+          results[k] = v
+        end
+      end
+      keyz.flatten.map { |k| results[k] }
+    end
+
+    def keys_by_node(*keyz)
+      keyz.flatten.inject({}) do |kbn, k|
+        node = node_for_key(k)
+        next if kbn[node] && kbn[node].include?(k)
+        kbn[node] ||= []
+        kbn[node] << k
+        kbn
+      end
+    end
+
+    def type(key)
+      method_missing(:type, key)
+    end
+  end
+end

metadata

@@ -5,8 +5,8 @@ version: !ruby/object:Gem::Version
   segments: 
   - 1
   - 0
-  - 5
-  version: 1.0.5
+  - 6
+  version: 1.0.6
 platform: ruby
 authors: 
 - Ezra Zygmuntowicz
@@ -19,7 +19,7 @@ autorequire: redis
 bindir: bin
 cert_chain: []
 
-date: 2010-04-21 00:00:00 -03:00
+date: 2010-04-26 00:00:00 -03:00
 default_executable: 
 dependencies: []
 
@@ -38,6 +38,7 @@ files:
 - lib/edis.rb
 - lib/redis/client.rb
 - lib/redis/dist_redis.rb
+- lib/redis/distributed.rb
 - lib/redis/hash_ring.rb
 - lib/redis/pipeline.rb
 - lib/redis/raketasks.rb