Skip to content

Commit

Permalink
Register agg functions with companion and overwrite
Browse files Browse the repository at this point in the history
  • Loading branch information
rui-mo committed Oct 18, 2023
1 parent 5d2ea9b commit bc20a87
Showing 1 changed file with 4 additions and 2 deletions.
6 changes: 4 additions & 2 deletions cpp/velox/operators/functions/RegistrationAllFunctions.cc
Original file line number Diff line number Diff line change
Expand Up @@ -58,8 +58,10 @@ void registerAllFunctions() {
// presto sql functions to overwrite the registration for same named functions.
velox::functions::prestosql::registerAllScalarFunctions();
velox::functions::sparksql::registerFunctions("");
velox::aggregate::prestosql::registerAllAggregateFunctions();
velox::functions::aggregate::sparksql::registerAggregateFunctions("");
velox::aggregate::prestosql::registerAllAggregateFunctions(
"", true /*registerCompanionFunctions*/, true /*overwrite*/);
velox::functions::aggregate::sparksql::registerAggregateFunctions(
"", true /*registerCompanionFunctions*/, true /*overwrite*/);
velox::window::prestosql::registerAllWindowFunctions();
velox::functions::window::sparksql::registerWindowFunctions("");
// Using function overwrite to handle function names mismatch between Spark and Velox.
Expand Down

0 comments on commit bc20a87

Please sign in to comment.