Merge pull request #120 from JuliaDatabases/fix/cluster-precompile-time

fix: precompilation and conflicting cluster _time

Author: jkaye2012

src/client.jl

@@ -27,7 +27,7 @@ const CLUSTER_MULTI_KEY_COMMANDS = Set([
     "eval", "evalsha",
 
     # Server commands - need to broadcast to all nodes
-    "flushall", "flushdb", "_time",
+    "flushall", "flushdb",
 
     # Pub/Sub commands - need special handling
     "publish"

src/cluster_commands.jl

@@ -705,29 +705,11 @@ function flushdb(cluster::RedisClusterConnection, db::Integer)
     return "OK"
 end
 
-"""
-    _time(cluster::RedisClusterConnection)
-
-Cluster version of TIME - returns time from a random node.
-Returns current Unix time from one of the cluster nodes.
-"""
-function _time(cluster::RedisClusterConnection)
-    if isempty(cluster.node_connections)
-        throw(ConnectionException("No active connections in cluster"))
-    end
-
-    # Get time from a random master node
-    connections = collect(values(cluster.node_connections))
-    random_conn = rand(connections)
-
-    response = execute_command(random_conn, ["TIME"])
-    return convert_response(Array{AbstractString,1}, response)
-end
-
 """
     time(cluster::RedisClusterConnection)
 
-Cluster version of TIME - returns DateTime from a random node.
+Cluster version of TIME - returns DateTime from a cluster node.
+Uses the `_time` method generated by `@redisfunction`.
 """
 function time(cluster::RedisClusterConnection)
     t = _time(cluster)

src/parser.jl

@@ -2,8 +2,6 @@
 Formatting of incoming Redis Replies
 """
 
-include("connection.jl")
-
 function getline(t::Transport.RedisTransport)
     l = chomp(Transport.read_line(t))
     length(l) > 1 || throw(ProtocolException("Invalid response received: $l"))