• judeng's avatar
    Fixes around AOF failed rewrite rate limiting (#10582) · d4cbd814
    judeng authored
    
    
    Changes:
    1. Check the failed rewrite time threshold only when we actually consider triggering a rewrite.
      i.e. this should be the last condition tested, since the test has side effects (increasing time threshold)
      Could have happened in some rare scenarios 
    2. no limit in startup state (e.g. after restarting redis that previously failed and had many incr files)
    3. the “triggered the limit” log would be recorded only when the limit status is returned
    4. remove failure count in log (could be misleading in some cases)
    Co-authored-by: default avatarchenyang8094 <chenyang8094@users.noreply.github.com>
    Co-authored-by: default avatarOran Agra <oran@redislabs.com>
    d4cbd814
aof.c 94.4 KB