Skip to content

Commit

Permalink
Rename redis in aof logs and proc title redis-aof-rewrite to valkey-a…
Browse files Browse the repository at this point in the history
…of-rewrite (#393)

Renamed redis to valkey/server in aof.c serverlogs.

The AOF rewrite child process title is set to "redis-aof-rewrite" if
Valkey was started from a redis-server symlink, otherwise to
"valkey-aof-rewrite".

This is a breaking changes since logs are changed.

Part of #207.

---------

Signed-off-by: Shivshankar-Reddy <shiva.sheri.github@gmail.com>
  • Loading branch information
Shivshankar-Reddy committed May 1, 2024
1 parent f4e10ee commit 8abeb79
Show file tree
Hide file tree
Showing 2 changed files with 12 additions and 8 deletions.
18 changes: 11 additions & 7 deletions src/aof.c
Expand Up @@ -996,7 +996,7 @@ int startAppendOnly(void) {

if (rewriteAppendOnlyFileBackground() == C_ERR) {
server.aof_state = AOF_OFF;
serverLog(LL_WARNING,"Redis needs to enable the AOF but can't trigger a background AOF rewrite operation. Check the above logs for more info about the error.");
serverLog(LL_WARNING,"The server needs to enable the AOF but can't trigger a background AOF rewrite operation. Check the above logs for more info about the error.");
return C_ERR;
}
}
Expand Down Expand Up @@ -1121,7 +1121,7 @@ void flushAppendOnlyFile(int force) {
/* Otherwise fall through, and go write since we can't wait
* over two seconds. */
server.aof_delayed_fsync++;
serverLog(LL_NOTICE,"Asynchronous AOF fsync is taking too long (disk is busy?). Writing the AOF buffer without waiting for fsync to complete, this may slow down Redis.");
serverLog(LL_NOTICE,"Asynchronous AOF fsync is taking too long (disk is busy?). Writing the AOF buffer without waiting for fsync to complete, this may slow down the server.");
}
}
/* We want to perform a single write. This should be guaranteed atomic
Expand Down Expand Up @@ -1183,7 +1183,7 @@ void flushAppendOnlyFile(int force) {
if (ftruncate(server.aof_fd, server.aof_last_incr_size) == -1) {
if (can_log) {
serverLog(LL_WARNING, "Could not remove short write "
"from the append-only file. Redis may refuse "
"from the append-only file. The server may refuse "
"to load the AOF the next time it starts. "
"ftruncate: %s", strerror(errno));
}
Expand Down Expand Up @@ -1224,7 +1224,7 @@ void flushAppendOnlyFile(int force) {
* OK state and log the event. */
if (server.aof_last_write_status == C_ERR) {
serverLog(LL_NOTICE,
"AOF write error looks solved, Redis can write again.");
"AOF write error looks solved. The server can write again.");
server.aof_last_write_status = C_OK;
}
}
Expand Down Expand Up @@ -1634,14 +1634,14 @@ int loadSingleAppendOnlyFile(char *filename) {
}
}
serverLog(LL_WARNING, "Unexpected end of file reading the append only file %s. You can: "
"1) Make a backup of your AOF file, then use ./redis-check-aof --fix <filename.manifest>. "
"1) Make a backup of your AOF file, then use ./valkey-check-aof --fix <filename.manifest>. "
"2) Alternatively you can set the 'aof-load-truncated' configuration option to yes and restart the server.", filename);
ret = AOF_FAILED;
goto cleanup;

fmterr: /* Format error. */
serverLog(LL_WARNING, "Bad file format reading the append only file %s: "
"make a backup of your AOF file, then use ./redis-check-aof --fix <filename.manifest>", filename);
"make a backup of your AOF file, then use ./valkey-check-aof --fix <filename.manifest>", filename);
ret = AOF_FAILED;
/* fall through to cleanup. */

Expand Down Expand Up @@ -2471,7 +2471,11 @@ int rewriteAppendOnlyFileBackground(void) {
char tmpfile[256];

/* Child */
serverSetProcTitle("redis-aof-rewrite");
if (strstr(server.exec_argv[0],"redis-server") != NULL) {
serverSetProcTitle("redis-aof-rewrite");
} else {
serverSetProcTitle("valkey-aof-rewrite");
}
serverSetCpuAffinity(server.aof_rewrite_cpulist);
snprintf(tmpfile,256,"temp-rewriteaof-bg-%d.aof", (int) getpid());
if (rewriteAppendOnlyFile(tmpfile) == C_OK) {
Expand Down
2 changes: 1 addition & 1 deletion src/cluster_legacy.c
Expand Up @@ -808,7 +808,7 @@ int clusterLockConfig(char *filename) {
* we need save `fd` to `cluster_config_file_lock_fd`, so that in serverFork(),
* it will be closed in the child process.
* If it is not closed, when the main process is killed -9, but the child process
* (redis-aof-rewrite) is still alive, the fd(lock) will still be held by the
* (valkey-aof-rewrite) is still alive, the fd(lock) will still be held by the
* child process, and the main process will fail to get lock, means fail to start. */
server.cluster_config_file_lock_fd = fd;
#else
Expand Down

0 comments on commit 8abeb79

Please sign in to comment.