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

…cubator#14)
rui-mo · Jan 11, 2023 · d895ba0 · d895ba0
1 parent 4607e1a
commit d895ba0
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 {