enso-org / dataframes

A library for working with tabular data in Luna.
https://luna-lang.org
MIT License
6 stars 5 forks source link

Allow using Length aggregate function with all types of columns #133

Closed mwu-tow closed 5 years ago

mwu-tow commented 5 years ago

@kustosz reported that Table.aggregate does not allow applying Length function onto column of type string. While generally aggregate functions were written with arithmetic types in mind, there's no reason not to allow Length working with any type of data. Length just counts elements in group, it doesn't care what element type is.

The check has been moved (we still need to prevent instantiation of numeric aggregator templates with strings) and additional function has been introduced to check if given aggregate function works with given column type.