1. 07 Jan, 2020 2 commits
  2. 06 Jan, 2020 1 commit
  3. 01 Jan, 2020 1 commit
    • antirez's avatar
      Fix active expire division by zero. · 0af467d1
      antirez authored
      Likely fix #6723.
      
      This is what happens AFAIK: we enter the main loop where we expire stuff
      until a given percentage of keys is still found to be logically expired.
      There are however other potential exit conditions.
      
      However the "sampled" variable is not always incremented inside the
      loop, because we may found no valid slot as we scan the hash table, but
      just NULLs ad dict entries. So when the do/while loop condition is
      triggered at the end, we do (expired*100/sampled), dividing by zero if
      we sampled 0 keys.
      0af467d1
  4. 29 Dec, 2019 2 commits
    • antirez's avatar
      Fix duplicated CLIENT SETNAME reply. · 6e4f70b8
      antirez authored
      Happened when we set the name to "" to cancel the name.
      Was introduced during the RESP3 refactoring.
      
      See #6036.
      6e4f70b8
    • antirez's avatar
      Inline protocol: handle empty strings well. · 5521910d
      antirez authored
      This bug is from the first version of Redis. Probably the problem here
      is that before we used an SDS split function that created empty strings
      for additional spaces, like in "SET    foo          bar".
      AFAIK later we replaced it with the curretn sdssplitarg() API that has
      no such a problem. As a result, we introduced a bug, where it is no
      longer possible to do something like:
      
          SET foo ""
      
      Using the inline protocol. Now it is fixed.
      5521910d
  5. 26 Dec, 2019 2 commits
    • Oran Agra's avatar
      config.c adjust config limits and mutable · 0c3fe52e
      Oran Agra authored
      - make lua-replicate-commands mutable (it never was, but i don't see why)
      - make tcp-backlog immutable (fix a recent refactory mistake)
      - increase the max limit of a few configs to match what they were before
      the recent refactory
      0c3fe52e
    • Guy Benoish's avatar
      Stream: Handle streamID-related edge cases · 1f75ce30
      Guy Benoish authored
      This commit solves several edge cases that are related to
      exhausting the streamID limits: We should correctly calculate
      the succeeding streamID instead of blindly incrementing 'seq'
      This affects both XREAD and XADD.
      
      Other (unrelated) changes:
      Reply with a better error message when trying to add an entry
      to a stream that has exhausted last_id
      1f75ce30
  6. 20 Dec, 2019 1 commit
  7. 18 Dec, 2019 6 commits
  8. 17 Dec, 2019 11 commits
  9. 16 Dec, 2019 1 commit
  10. 13 Dec, 2019 1 commit
  11. 12 Dec, 2019 5 commits
  12. 11 Dec, 2019 2 commits
  13. 09 Dec, 2019 2 commits
  14. 05 Dec, 2019 2 commits
    • Eran Liberty's avatar
      - memcpy(&id,ri.key,ri.key_len); · 08c3fe80
      Eran Liberty authored
      +        memcpy(&id,ri.key,sizeof(id));
      
      The memcpy from the key to the id reliease on the fact that this key
      *should* be 8 bytes long as it was entered as such a few lines up the
      code.
      
      BUT if someone will change the code to the point this is no longer true,
      current code can trash the stack which makes debugging very hard
      while this fix will result in some garbage id, or even page fault.
      Both are preferable to stack mangaling.
      08c3fe80
    • antirez's avatar
      2271cf03
  15. 03 Dec, 2019 1 commit