Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

update valkey in serverLog messeges in server.c file #231

Merged

Conversation

Shivshankar-Reddy
Copy link
Contributor

@Shivshankar-Reddy Shivshankar-Reddy commented Apr 5, 2024

Updated keyword "Redis" to "Valkey" in log messeges in server.c file

Signed-off-by: Shivshankar-Reddy <shiva.sheri.github@gmail.com>
@Shivshankar-Reddy
Copy link
Contributor Author

@zuiderkwast Can you review this PR?

@zuiderkwast zuiderkwast added the breaking-change Indicates a possible backwards incompatible change label Apr 11, 2024
@zuiderkwast
Copy link
Contributor

Partial implementation of #207.

Log messages are breaking changes. Let's remember for release notes.

Copy link
Contributor

@zuiderkwast zuiderkwast left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

OK, LGTM

return 0;
#else
if (serverCommunicateSystemd("STATUS=Redis is loading...\n") <= 0)
if (serverCommunicateSystemd("STATUS=Valkey is loading...\n") <= 0)
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is a special message. Not a normal log entry. I'm marking it for special attention.

@hwware hwware merged commit 4be97eb into valkey-io:unstable Apr 11, 2024
14 checks passed
@zuiderkwast
Copy link
Contributor

Oops, maybe we shouldn't merge this after the last discussions in #207.

No problem. We can just update these again.

zuiderkwast pushed a commit to zuiderkwast/placeholderkv that referenced this pull request Apr 17, 2024
Updated keyword "Redis" to "Valkey" in log messeges in server.c file

Signed-off-by: Shivshankar-Reddy <shiva.sheri.github@gmail.com>
@Shivshankar-Reddy Shivshankar-Reddy deleted the update-server-serevrLog branch April 18, 2024 16:56
PatrickJS pushed a commit to PatrickJS/placeholderkv that referenced this pull request Apr 24, 2024
Updated keyword "Redis" to "Valkey" in log messeges in server.c file

Signed-off-by: Shivshankar-Reddy <shiva.sheri.github@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
breaking-change Indicates a possible backwards incompatible change
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants