Skip to content

fix maxmemory config warning#6845

Merged
antirez merged 1 commit intoredis:unstablefrom
oranagra:maxmemory_warning
Feb 6, 2020
Merged

fix maxmemory config warning#6845
antirez merged 1 commit intoredis:unstablefrom
oranagra:maxmemory_warning

Conversation

@oranagra
Copy link
Member

@oranagra oranagra commented Feb 6, 2020

the warning condition was if usage > limit (saying it'll cause eviction
or oom), but in fact the eviction and oom depends on used minus slave
buffers.

other than fixing the condition, i add info about the current usage and
limit, which may be useful when looking at the log.

the warning condition was if usage > limit (saying it'll cause eviction
or oom), but in fact the eviction and oom depends on used minus slave
buffers.

other than fixing the condition, i add info about the current usage and
limit, which may be useful when looking at the log.
@antirez antirez merged commit be52082 into redis:unstable Feb 6, 2020
@antirez
Copy link
Contributor

antirez commented Feb 6, 2020

Thank you.

JackieXie168 pushed a commit to JackieXie168/redis that referenced this pull request Feb 20, 2020
oshadmi added a commit to oshadmi/redis that referenced this pull request Sep 28, 2025
$ gitlog v8.2.1..v8.2.5

redis#6867 Improve BUILD_INTEL_SVS_OPT flag validation
redis#6845 Fix vector compression type reporting and rename SVS environment variable
redis#6794 Fix rpcountFree casting
redis#6825 Bump vecsim 8.2.4
redis#6787 Fix deadlock while RDB loading and RM_Yield (redis#6763)
redis#6723 Fix ACLUserMayAccessIndex
redis#6701 Fix fd leak when OOM
redis#6676 bump svs version
redis#6671 Add SVS-VAMANA index to info fields telemetry
redis#6665 Optimize rs_wall_clock_diff_ns by removing redundant if branch
redis#6641 Update vectors memory in total index info memory
redis#6634 Log DocTable capacity growth for memory diagnostics
redis#6649 Bump minimal redis version
redis#6648 Fix "has map" performance
redis#6645 Trie: hide expensive memory usage computation behind a test-specific feature flag.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants