Skip to content

Pipeliner expose queued commands#3496

Merged
htemelski-oss merged 2 commits into
redis:masterfrom
cxljs:pipeline
Sep 1, 2025
Merged

Pipeliner expose queued commands#3496
htemelski-oss merged 2 commits into
redis:masterfrom
cxljs:pipeline

Conversation

@cxljs

@cxljs cxljs commented Aug 31, 2025

Copy link
Copy Markdown
Contributor

closes #3489

@ndyakov

ndyakov commented Sep 1, 2025

Copy link
Copy Markdown
Member

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

@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.

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

Comment thread pipeline.go
htemelski-oss
htemelski-oss previously approved these changes Sep 1, 2025

@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.

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-oss htemelski-oss 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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Expose pipeline command results via Cmds() method

3 participants