Skip to content

Commit

Permalink
Include decimal in supported types for range window order-by columns (#…
Browse files Browse the repository at this point in the history
…11710)

As a follow-up to #11645, this change includes `DECIMAL` among the list of data-types that may be used in the order-by column for `RANGE`-based window functions, listed in the exception message.

Authors:
  - MithunR (https://github.com/mythrocks)

Approvers:
  - Gera Shegalov (https://github.com/gerashegalov)
  - Nghia Truong (https://github.com/ttnghia)

URL: #11710
  • Loading branch information
mythrocks authored Sep 19, 2022
1 parent 75d126a commit 972708a
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion java/src/main/java/ai/rapids/cudf/Table.java
Original file line number Diff line number Diff line change
Expand Up @@ -3922,7 +3922,7 @@ public Table aggregateWindowsOverRanges(AggregationOverWindow... windowAggregate
break;
default:
throw new IllegalArgumentException("Expected range-based window orderBy's " +
"type: integral (Boolean-exclusive) and timestamp");
"type: integral (Boolean-exclusive), decimal, and timestamp");
}

ColumnWindowOps ops = groupedOps.computeIfAbsent(agg.getColumnIndex(), (idx) -> new ColumnWindowOps());
Expand Down

0 comments on commit 972708a

Please sign in to comment.