1. 09 Jun, 2020 8 commits
  2. 06 Jun, 2020 6 commits
  3. 28 May, 2020 18 commits
  4. 25 May, 2020 6 commits
    • antirez's avatar
      Clarify what is happening in PR #7320. · 2e591fc4
      antirez authored
      2e591fc4
    • zhaozhao.zz's avatar
      PSYNC2: second_replid_offset should be real meaningful offset · cbb51fb8
      zhaozhao.zz authored
      After adjustMeaningfulReplOffset(), all the other related variable
      should be updated, including server.second_replid_offset.
      
      Or the old version redis like 5.0 may receive wrong data from
      replication stream, cause redis 5.0 can sync with redis 6.0,
      but doesn't know meaningful offset.
      cbb51fb8
    • Oran Agra's avatar
      add CI for 32bit build · e0fc88b4
      Oran Agra authored
      e0fc88b4
    • antirez's avatar
      Make disconnectSlaves() synchronous in the base case. · e3f864b5
      antirez authored
      Otherwise we run into that:
      
      Backtrace:
      src/redis-server 127.0.0.1:21322(logStackTrace+0x45)[0x479035]
      src/redis-server 127.0.0.1:21322(sigsegvHandler+0xb9)[0x4797f9]
      /lib/x86_64-linux-gnu/libpthread.so.0(+0x11390)[0x7fd373c5e390]
      src/redis-server 127.0.0.1:21322(_serverAssert+0x6a)[0x47660a]
      src/redis-server 127.0.0.1:21322(freeReplicationBacklog+0x42)[0x451282]
      src/redis-server 127.0.0.1:21322[0x4552d4]
      src/redis-server 127.0.0.1:21322[0x4c5593]
      src/redis-server 127.0.0.1:21322(aeProcessEvents+0x2e6)[0x42e786]
      src/redis-server 127.0.0.1:21322(aeMain+0x1d)[0x42eb0d]
      src/redis-server 127.0.0.1:21322(main+0x4c5)[0x42b145]
      /lib/x86_64-linux-gnu/libc.so.6(__libc_start_main+0xf0)[0x7fd3738a3830]
      src/redis-server 127.0.0.1:21322(_start+0x29)[0x42b409]
      
      Since we disconnect all the replicas and free the replication backlog in
      certain replication paths, and the code that will free the replication
      backlog expects that no replica is connected.
      
      However we still need to free the replicas asynchronously in certain
      cases, as documented in the top comment of disconnectSlaves().
      e3f864b5
    • ShooterIT's avatar
      Implements sendfile for redis. · 8af1e513
      ShooterIT authored
      8af1e513
    • antirez's avatar
      Fix #7306 less aggressively. · 3c21418c
      antirez authored
      Citing from the issue:
      
      btw I suggest we change this fix to something else:
      * We revert the fix.
      * We add a call that disconnects chained replicas in the place where we trim the replica (that is a master i this case) offset.
      This way we can avoid disconnections when there is no trimming of the backlog.
      
      Note that we now want to disconnect replicas asynchronously in
      disconnectSlaves(), because it's in general safer now that we can call
      it from freeClient(). Otherwise for instance the command:
      
          CLIENT KILL TYPE master
      
      May crash: clientCommand() starts running the linked of of clients,
      looking for clients to kill. However it finds the master, kills it
      calling freeClient(), but this in turn calls replicationCacheMaster()
      that may also call disconnectSlaves() now. So the linked list iterator
      of the clientCommand() will no longer be valid.
      3c21418c
  5. 22 May, 2020 2 commits