Skip to content

Rename Redis to Valkey in the crash report.#253

Closed
zhulipeng wants to merge 3 commits intovalkey-io:unstablefrom
zhulipeng:crash-report
Closed

Rename Redis to Valkey in the crash report.#253
zhulipeng wants to merge 3 commits intovalkey-io:unstablefrom
zhulipeng:crash-report

Conversation

@zhulipeng
Copy link
Member

@zhulipeng zhulipeng commented Apr 8, 2024

This patch try to correct the server crash report.
Rename Redis to Valkey.

Copy link
Member

@madolson madolson left a comment

Choose a reason for hiding this comment

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

I totally forgot about this, this is another thing to backport.

Signed-off-by: Lipeng Zhu <[email protected]>
@enjoy-binbin enjoy-binbin changed the title Update Valkey crash report. Rename Redis to Valkey in the crash report. Apr 8, 2024
@zhulipeng
Copy link
Member Author

Resolved in #263.

@zhulipeng zhulipeng closed this Apr 9, 2024
@madolson
Copy link
Member

madolson commented Apr 9, 2024

@lipzhu I would keep this open. We may still want to apply the changed on REDIS BUG REPORT END. Make sure to include from START to END that this PR implements.

@zhulipeng zhulipeng reopened this Apr 9, 2024
@zhulipeng
Copy link
Member Author

@lipzhu I would keep this open. We may still want to apply the changed on REDIS BUG REPORT END. Make sure to include from START to END that this PR implements.

Ok, reopen the PR and solve the conflicts.

@madolson madolson added the breaking-change Indicates a possible backwards incompatible change label Apr 9, 2024
@madolson
Copy link
Member

madolson commented Apr 9, 2024

Thanks, we need to decide if this is a breaking change, putting it in the 8.0 project for the time being.

@zuiderkwast
Copy link
Contributor

This is a potential candidate for configurable redis compat mode. #274

@zhulipeng
Copy link
Member Author

Solved in #306.

@zhulipeng zhulipeng closed this Apr 23, 2024
@zhulipeng zhulipeng deleted the crash-report branch May 21, 2024 07:17
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

Status: Done

Development

Successfully merging this pull request may close these issues.

4 participants