1. 12 2月, 2013 4 次提交
    • A
      PSYNC: work in progress, preview #2, rebased to unstable. · 700e5eb4
      antirez 提交于
      700e5eb4
    • A
      Use the new unified protocol to send SELECT to slaves. · 4d8655cf
      antirez 提交于
      SELECT was still transmitted to slaves using the inline protocol, that
      is conceived mostly for humans to type into telnet sessions, and is
      notably not understood by redis-cli --slave.
      
      Now the new protocol is used instead.
      4d8655cf
    • A
      Use replicationFeedSlaves() to send PING to slaves. · 01c21f99
      antirez 提交于
      A Redis master sends PING commands to slaves from time to time: doing
      this ensures that even if absence of writes, the master->slave channel
      remains active and the slave can feel the master presence, instead of
      closing the connection for timeout.
      
      This commit changes the way PINGs are sent to slaves in order to use the
      standard interface used to replicate all the other commands, that is,
      the function replicationFeedSlaves().
      
      With this change the stream of commands sent to every slave is exactly
      the same regardless of their exact state (Transferring RDB for first
      synchronization or slave already online). With the previous
      implementation the PING was only sent to online slaves, with the result
      that the output stream from master to slaves was not identical for all
      the slaves: this is a problem if we want to implement partial resyncs in
      the future using a global replication stream offset.
      
      TL;DR: this commit should not change the behaviour in practical terms,
      but is just something in preparation for partial resynchronization
      support.
      01c21f99
    • A
      Emit SELECT to slaves in a centralized way. · 5a35e485
      antirez 提交于
      Before this commit every Redis slave had its own selected database ID
      state. This was not actually useful as the emitted stream of commands
      is identical for all the slaves.
      
      Now the the currently selected database is a global state that is set to
      -1 when a new slave is attached, in order to force the SELECT command to
      be re-emitted for all the slaves.
      
      This change is useful in order to implement replication partial
      resynchronization in the future, as makes sure that the stream of
      commands received by slaves, including SELECT commands, are exactly the
      same for every slave connected, at any time.
      
      In this way we could have a global offset that can identify a specific
      piece of the master -> slaves stream of commands.
      5a35e485
  2. 11 2月, 2013 6 次提交
  3. 08 2月, 2013 2 次提交
  4. 05 2月, 2013 3 次提交
    • A
      TCP_NODELAY after SYNC: changes to the implementation. · 5f7dff4d
      antirez 提交于
      5f7dff4d
    • C
      Turn off TCP_NODELAY on the slave socket after SYNC. · 1d80acae
      charsyam 提交于
      Further details from @antirez:
      
      It was reported by @StopForumSpam on Twitter that the Redis replication
      link was strangely using multiple TCP packets for multiple commands.
      This wastes a lot of bandwidth and is due to the TCP_NODELAY option we
      enable on the socket after accepting a new connection.
      
      However the master -> slave channel is a one-way channel since Redis
      replication is asynchronous, so there is no point in trying to reduce
      the latency, we should aim to reduce the bandwidth. For this reason this
      commit introduces the ability to disable the nagle algorithm on the
      socket after a successful SYNC.
      
      This feature is off by default because the delay can be up to 40
      milliseconds with normally configured Linux kernels.
      1d80acae
    • R
      retval doesn't initalized · 2543fa64
      Rock Li 提交于
      If each if conditions are all fail, variable retval will under uninitlized
      2543fa64
  5. 04 2月, 2013 1 次提交
    • G
      Fix a bug in srandmemberWithCountCommand() · 57e411c6
      Gengliang Wang 提交于
      In CASE 2, the call sunionDiffGenericCommand will involve the string "srandmember"
      > sadd foo one
      (integer 1)
      > sadd srandmember two
      (integer 2)
      > srandmember foo 3
      1)"one"
      2)"two"
      57e411c6
  6. 29 1月, 2013 2 次提交
  7. 28 1月, 2013 11 次提交
  8. 22 1月, 2013 2 次提交
    • A
      redis-cli --bigkeys output is now simpler to understand. · 767a53aa
      antirez 提交于
      767a53aa
    • A
      UNSUBSCRIBE and PUNSUBSCRIBE: always provide a reply. · 3ff75e58
      antirez 提交于
      UNSUBSCRIBE and PUNSUBSCRIBE commands are designed to mass-unsubscribe
      the client respectively all the channels and patters if called without
      arguments.
      
      However when these functions are called without arguments, but there are
      no channels or patters we are subscribed to, the old behavior was to
      don't reply at all.
      
      This behavior is broken, as every command should always reply.
      Also it is possible that we are no longer subscribed to a channels but we
      are subscribed to patters or the other way around, and the client should
      be notified with the correct number of subscriptions.
      
      Also it is not pretty that sometimes we did not receive a reply at all
      in a redis-cli session from these commands, blocking redis-cli trying
      to read the reply.
      
      This fixes issue #714.
      3ff75e58
  9. 21 1月, 2013 3 次提交
  10. 19 1月, 2013 6 次提交
    • A
      Additionally two typos fixed thanks to @jodal · 635c532c
      antirez 提交于
      635c532c
    • A
      Whitelist SIGUSR1 to avoid auto-triggering errors. · 39f0a33f
      antirez 提交于
      This commit fixes issue #875 that was caused by the following events:
      
      1) There is an active child doing BGSAVE.
      2) flushall is called (or any other condition that makes Redis killing
      the saving child process).
      3) An error is sensed by Redis as the child exited with an error (killed
      by a singal), that stops accepting write commands until a BGSAVE happens
      to be executed with success.
      
      Whitelisting SIGUSR1 and making sure Redis always uses this signal in
      order to kill its own children fixes the issue.
      39f0a33f
    • A
      Clear server.shutdown_asap on failed shutdown. · 1e20c939
      antirez 提交于
      When a SIGTERM is received Redis schedules a shutdown. However if it
      fails to perform the shutdown it must be clear the shutdown_asap flag
      otehrwise it will try again and again possibly making the server
      unusable.
      1e20c939
    • A
      Slowlog: don't log EXEC but just the executed commands. · d766907c
      antirez 提交于
      The Redis Slow Log always used to log the slow commands executed inside
      a MULTI/EXEC block. However also EXEC was logged at the end, which is
      perfectly useless.
      
      Now EXEC is no longer logged and a test was added to test this behavior.
      
      This fixes issue #759.
      d766907c
    • G
      Fixed many typos. · 1caf0939
      guiquanz 提交于
      Conflicts fixed, mainly because 2.8 has no cluster support / files:
      	00-RELEASENOTES
      	src/cluster.c
      	src/crc16.c
      	src/redis-trib.rb
      	src/redis.h
      1caf0939
    • C
      redis-cli prompt bug fix · ff1e4d22
      charsyam 提交于
      ff1e4d22