From ed95d09c36526f7154b21fdf18b8b42103f1395d Mon Sep 17 00:00:00 2001 From: Carlo Piovesan Date: Mon, 13 Nov 2023 13:28:31 +0100 Subject: [PATCH] DecimalQuantiles: Move QuantileSerializationType to separate header Previous one was autogenerated --- src/common/enum_util.cpp | 2 +- .../aggregate/holistic/quantile.cpp | 1 + .../aggregate/holistic_functions.hpp | 8 ------- .../aggregate/quantile_enum.hpp | 21 +++++++++++++++++++ 4 files changed, 23 insertions(+), 9 deletions(-) create mode 100644 src/include/duckdb/core_functions/aggregate/quantile_enum.hpp diff --git a/src/common/enum_util.cpp b/src/common/enum_util.cpp index 246d4f37aff0..64fa25994338 100644 --- a/src/common/enum_util.cpp +++ b/src/common/enum_util.cpp @@ -64,7 +64,7 @@ #include "duckdb/common/types/timestamp.hpp" #include "duckdb/common/types/vector.hpp" #include "duckdb/common/types/vector_buffer.hpp" -#include "duckdb/core_functions/aggregate/holistic_functions.hpp" +#include "duckdb/core_functions/aggregate/quantile_enum.hpp" #include "duckdb/execution/index/art/art.hpp" #include "duckdb/execution/index/art/node.hpp" #include "duckdb/execution/operator/scan/csv/base_csv_reader.hpp" diff --git a/src/core_functions/aggregate/holistic/quantile.cpp b/src/core_functions/aggregate/holistic/quantile.cpp index eb0fba965733..324aefa3bddd 100644 --- a/src/core_functions/aggregate/holistic/quantile.cpp +++ b/src/core_functions/aggregate/holistic/quantile.cpp @@ -1,5 +1,6 @@ #include "duckdb/execution/expression_executor.hpp" #include "duckdb/core_functions/aggregate/holistic_functions.hpp" +#include "duckdb/core_functions/aggregate/quantile_enum.hpp" #include "duckdb/planner/expression.hpp" #include "duckdb/common/operator/cast_operators.hpp" #include "duckdb/common/operator/abs.hpp" diff --git a/src/include/duckdb/core_functions/aggregate/holistic_functions.hpp b/src/include/duckdb/core_functions/aggregate/holistic_functions.hpp index 16f21dceb2a4..3627874e86a8 100644 --- a/src/include/duckdb/core_functions/aggregate/holistic_functions.hpp +++ b/src/include/duckdb/core_functions/aggregate/holistic_functions.hpp @@ -84,12 +84,4 @@ struct ReservoirQuantileFun { static AggregateFunctionSet GetFunctions(); }; -enum class QuantileSerializationType : uint8_t { - NON_DECIMAL = 0, - DECIMAL_DISCRETE, - DECIMAL_DISCRETE_LIST, - DECIMAL_CONTINUOUS, - DECIMAL_CONTINUOUS_LIST -}; - } // namespace duckdb diff --git a/src/include/duckdb/core_functions/aggregate/quantile_enum.hpp b/src/include/duckdb/core_functions/aggregate/quantile_enum.hpp new file mode 100644 index 000000000000..161f1a90eddf --- /dev/null +++ b/src/include/duckdb/core_functions/aggregate/quantile_enum.hpp @@ -0,0 +1,21 @@ +//===----------------------------------------------------------------------===// +// DuckDB +// +// duckdb/core_functions/aggregate/quantile_enum.hpp +// +// +//===----------------------------------------------------------------------===// + +#pragma once + +namespace duckdb { + +enum class QuantileSerializationType : uint8_t { + NON_DECIMAL = 0, + DECIMAL_DISCRETE, + DECIMAL_DISCRETE_LIST, + DECIMAL_CONTINUOUS, + DECIMAL_CONTINUOUS_LIST +}; + +}