[OPPRO-104] Support more cases of filter and its pushdown (facebookin…

…cubator#14)
PHILO-HE · Jan 31, 2023 · 13040ff · 13040ff
1 parent 5382867
commit 13040ff
Show file tree

Hide file tree

Showing 18 changed files with 2,074 additions and 311 deletions.
diff --git a/velox/functions/prestosql/aggregates/CMakeLists.txt b/velox/functions/prestosql/aggregates/CMakeLists.txt
@@ -33,6 +33,7 @@ add_library(
   MapAggregateBase.cpp
   MapUnionAggregate.cpp
   MinMaxAggregates.cpp
+  MinMaxAggregates.h
   MinMaxByAggregates.cpp
   CountAggregate.cpp
   CountAggregate.h

diff --git a/velox/functions/prestosql/aggregates/MinMaxAggregates.cpp b/velox/functions/prestosql/aggregates/MinMaxAggregates.cpp
@@ -14,13 +14,8 @@
  * limitations under the License.
  */
 
-#include <limits>
-#include "velox/exec/Aggregate.h"
-#include "velox/exec/AggregationHook.h"
-#include "velox/expression/FunctionSignature.h"
+#include "velox/functions/prestosql/aggregates/MinMaxAggregates.h"
 #include "velox/functions/prestosql/aggregates/AggregateNames.h"
-#include "velox/functions/prestosql/aggregates/SimpleNumericAggregate.h"
-#include "velox/functions/prestosql/aggregates/SingleValueAccumulator.h"
 
 namespace facebook::velox::aggregate::prestosql {