This repository has been archived by the owner on Nov 1, 2024. It is now read-only.
Making BaseColumn::genericUnaryUDF and the family free functions #197
Labels
good first issue
Good for newcomers
https://github.com/facebookresearch/torcharrow/blob/main/csrc/velox/column.h#L364-L377
This is in the Eager Mode/Velox Backend.
The generic UDF call methods should be general enough to not be bound to any columns. For example when there are no arguments or all arguments are scalars, conceptually this call is not bound to any columns. Technically they are static functions so don't require users to bind the calls to any column, but having these functions in the BaseColumn class makes them "sound" like being bound to BaseColumn. This is mostly a cosmetic change for new joiners to start with. More work will be required to truly make a UDF call interface that allows any forms of inputs.
Let's move these functions out of the BaseColumn class and make them free
The text was updated successfully, but these errors were encountered: