Skip to content

Commit ebb888d

Browse files
committed
fix: Load SELECT fields when using date functions in FT.AGGREGATE
When queries use date functions (YEAR, MONTH, etc.) alongside regular fields in SELECT, the translator now properly loads those fields using LOAD clause. Before: FT.AGGREGATE events "*" APPLY year(@created_at) AS year After: FT.AGGREGATE events "*" LOAD 1 name APPLY year(@created_at) AS year
1 parent 1f3d773 commit ebb888d

1 file changed

Lines changed: 4 additions & 0 deletions

File tree

sql_redis/translator.py

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -252,6 +252,10 @@ def _build_aggregate(
252252
load_fields.add(agg.field)
253253
for field_name in analyzed.groupby_fields:
254254
load_fields.add(field_name)
255+
# Load explicit SELECT fields when using date functions
256+
for field_name in parsed.fields:
257+
if field_name != "*":
258+
load_fields.add(field_name)
255259

256260
if load_fields:
257261
args.append("LOAD")

0 commit comments

Comments
 (0)