DEV: (cmds) FT.AGGREGATE - add COLLECT reducer#3320
Conversation
🛡️ Jit Security Scan Results✅ No security findings were detected in this PR
Security scan by Jit
|
There was a problem hiding this comment.
Cursor Bugbot has reviewed your changes and found 1 potential issue.
❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, have a team admin enable autofix in the Cursor dashboard.
Reviewed by Cursor Bugbot for commit 05f82f9. Configure here.
|
TYVM, @andy-stark-redis! |
|
|
||
| **Description** | ||
|
|
||
| Fetch complete documents or project specific fields directly within an aggregation query, with optional sorting, limiting, and deduplication per group. |
There was a problem hiding this comment.
I’m not sure whether Fetch is the right term here. Since you’re already familiar with the issue I mentioned in the other comments, I’ll leave it to your judgment.
Itzikvaknin
left a comment
There was a problem hiding this comment.
Looks good!
I left a few comments to improve clarity and correctness.
|
Thank you, @Itzikvaknin! |
|
@Itzikvaknin Also need |



Redis 8.8 feature
@Itzikvaknin I couldn't add you directly as a docs reviewer, but I'm hoping you can review as a guest (just leave a comment).
Note
Low Risk
Documentation-only changes; primary risk is inaccuracies in the new
COLLECTsyntax/semantics or generated diagram updates confusing users.Overview
Adds documentation for the new
REDUCE COLLECTreducer inFT.AGGREGATE(Redis 8.8.0), including the formal argument/schema definition, detailed reducer reference, and a new end-to-end example for collecting top documents per group.Expands the
FT.AGGREGATEcommand docs with additional argument summaries and a much more explicitAPPLYexpression/function grammar, and updates theft.aggregaterailroad diagram (static/images/railroad/ft.aggregate.svg) to reflect the new syntax.Reviewed by Cursor Bugbot for commit ffd9ebd. Bugbot is set up for automated code reviews on this repo. Configure here.