Skip to content

Conversation

cxljs
Copy link
Contributor

@cxljs cxljs commented Aug 31, 2025

closes #3489

@ndyakov
Copy link
Member

ndyakov commented Sep 1, 2025

I think this will be okay to have. Will let @htemelski-redis review.

Copy link
Member

@ndyakov ndyakov left a comment

Choose a reason for hiding this comment

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

Requested tests, not a blocker since this is straightforward getter.

htemelski-redis
htemelski-redis previously approved these changes Sep 1, 2025
Copy link
Member

@ndyakov ndyakov left a comment

Choose a reason for hiding this comment

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

Thank you, @cxljs!
@htemelski-redis you can review and merge if you are satisfied with the change.

Signed-off-by: Xiaolong Chen <fukua95@gmail.com>
Signed-off-by: Xiaolong Chen <fukua95@gmail.com>
@htemelski-redis htemelski-redis merged commit fafec3f into redis:master Sep 1, 2025
20 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

3 participants