gvsafronov/futriix
1
0
Fork 0
Code Issues Pull Requests Actions 1 Packages Projects Releases Wiki Activity

do not add save parameter during config rewrite in sentinel mode (#7945)

Browse Source 
Previous code would have added default redis save parameters
to the config file on rewrite, which would have been silently ignored
when the config file is loaded.

The new code avoids adding this, and also actively removes these lines
If added by a previous config rewrite.
This commit is contained in:
Wen Hui 2020-10-22 12:47:32 -04:00 committed by GitHub
parent 44bcbed2ee
commit f61435fe8b
1 changed files with 6 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
src/config.c
View File

@ -1335,6 +1335,12 @@ void rewriteConfigSaveOption(struct rewriteConfigState *state) {
int j; int j;
sds line; sds line;
/* In Sentinel mode we don't need to rewrite the save parameters */
if (server.sentinel_mode) {
rewriteConfigMarkAsProcessed(state,"save");
return;
}
/* Note that if there are no save parameters at all, all the current /* Note that if there are no save parameters at all, all the current
* config line with "save" will be detected as orphaned and deleted, * config line with "save" will be detected as orphaned and deleted,
* resulting into no RDB persistence as expected. */ * resulting into no RDB persistence as expected. */