Skip to content

feat(time-series): add multi-aggregation range APIs#3249

Open
nkaradzhov wants to merge 4 commits intoredis:masterfrom
nkaradzhov:ts-multiple-aggregators
Open

feat(time-series): add multi-aggregation range APIs#3249
nkaradzhov wants to merge 4 commits intoredis:masterfrom
nkaradzhov:ts-multiple-aggregators

Conversation

@nkaradzhov
Copy link
Copy Markdown
Collaborator

@nkaradzhov nkaradzhov commented Apr 23, 2026

Description

Describe your pull request here


Checklist

  • Does npm test pass with this change (including linting)?
  • Is the new or changed code fully tested?
  • Is a documentation update included (if this change modifies existing APIs, or introduces new ones)?

Note

Medium Risk
Adds new RedisTimeSeries range command variants and reply transformers, which could affect client parsing/typing and output shapes for TS.RANGE/MRANGE-style calls. Changes are read-only and well-tested, but touch RESP2/RESP3 transformations and command argument construction.

Overview
Adds new *MultiAggr TimeSeries commands that allow requesting multiple aggregations in a single range query, returning samples as { timestamp, values: [...] } for TS.RANGE/TS.REVRANGE and multi-key TS.MRANGE/TS.MREVRANGE variants (including WITHLABELS and SELECTED_LABELS).

Introduces shared argument parsing (TsRangeMultiAggrOptions / parseRangeMultiArguments) and new RESP2/RESP3 reply transformers for multi-aggregation samples, wires the commands into the public exports, and updates the README to use TIME_SERIES_AGGREGATION_TYPE plus document the new APIs.

Reviewed by Cursor Bugbot for commit 1479e42. Bugbot is set up for automated code reviews on this repo. Configure here.

@nkaradzhov nkaradzhov marked this pull request as ready for review April 23, 2026 08:25
@nkaradzhov nkaradzhov requested a review from PavelPashov April 23, 2026 08:26
Comment thread packages/time-series/README.md Outdated
Copy link
Copy Markdown

@cursor cursor Bot left a comment

Choose a reason for hiding this comment

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

Cursor Bugbot has reviewed your changes and found 1 potential issue.

Fix All in Cursor

Reviewed by Cursor Bugbot for commit a17df68. Configure here.

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.

2 participants