Skip to content

Mark time as keyless command#3722

Merged
ndyakov merged 1 commit into
redis:masterfrom
fatal10110:fix_time_keyless
Feb 26, 2026
Merged

Mark time as keyless command#3722
ndyakov merged 1 commit into
redis:masterfrom
fatal10110:fix_time_keyless

Conversation

@fatal10110

@fatal10110 fatal10110 commented Feb 24, 2026

Copy link
Copy Markdown
Contributor

closes #3721

@jit-ci

jit-ci Bot commented Feb 24, 2026

Copy link
Copy Markdown

Hi, I’m Jit, a friendly security platform designed to help developers build secure applications from day zero with an MVS (Minimal viable security) mindset.

In case there are security findings, they will be communicated to you as a comment inside the PR.

Hope you’ll enjoy using Jit.

Questions? Comments? Want to learn more? Get in touch with us.

@fatal10110 fatal10110 changed the title Mark time ask keyless command Mark time as keyless command Feb 24, 2026
@fatal10110

Copy link
Copy Markdown
Contributor Author

its unclear why CI is failing, and how is it related to this one line of code

@ndyakov

ndyakov commented Feb 25, 2026

Copy link
Copy Markdown
Member

@fatal10110 I will rerun the CI, it is possible we have a flaky test after merging #3705

@ndyakov ndyakov self-requested a review February 25, 2026 13:45

@ndyakov ndyakov left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

TIME is in fact keyless command.

@ndyakov ndyakov added the bug label Feb 26, 2026
@ndyakov ndyakov merged commit 5c21658 into redis:master Feb 26, 2026
77 of 80 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Time command inside multi with "keyed" commands returns CROSSLOT error

2 participants