GH-14866: [C++] Remove internal GroupBy implementation (#14867)

* Closes: #14866 Authored-by: Weston Pace <weston.pace@gmail.com> Signed-off-by: Weston Pace <weston.pace@gmail.com>
apache · Feb 21, 2023 · 92d91f5 · 92d91f5
1 parent a988302
commit 92d91f5
Show file tree

Hide file tree

Showing 16 changed files with 1,323 additions and 1,327 deletions.
diff --git a/cpp/src/arrow/CMakeLists.txt b/cpp/src/arrow/CMakeLists.txt
@@ -391,7 +391,7 @@ if(ARROW_COMPUTE)
        compute/api_vector.cc
        compute/cast.cc
        compute/exec.cc
-       compute/exec/aggregate.cc
+       compute/exec/groupby.cc
        compute/exec/accumulation_queue.cc
        compute/exec/aggregate_node.cc
        compute/exec/asof_join_node.cc

diff --git a/cpp/src/arrow/compute/api.h b/cpp/src/arrow/compute/api.h
@@ -58,3 +58,4 @@
 
 #include "arrow/compute/exec.h"            // IWYU pragma: export
 #include "arrow/compute/exec/exec_plan.h"  // IWYU pragma: export
+#include "arrow/compute/exec/groupby.h"    // IWYU pragma: export
diff --git a/cpp/src/arrow/compute/exec/CMakeLists.txt b/cpp/src/arrow/compute/exec/CMakeLists.txt
@@ -52,6 +52,7 @@ add_arrow_compute_test(asof_join_node_test
                        test_nodes.cc)
 add_arrow_compute_test(tpch_node_test PREFIX "arrow-compute")
 add_arrow_compute_test(union_node_test PREFIX "arrow-compute")
+add_arrow_compute_test(groupby_test PREFIX "arrow-compute")
 add_arrow_compute_test(util_test
                        PREFIX
                        "arrow-compute"

diff --git a/cpp/src/arrow/compute/exec/aggregate.cc b/cpp/src/arrow/compute/exec/aggregate.cc
diff --git a/cpp/src/arrow/compute/exec/aggregate.h b/cpp/src/arrow/compute/exec/aggregate.h
Original file line number	Diff line number	Diff line change
Expand Up		@@ -58,3 +58,4 @@

		#include "arrow/compute/exec.h" // IWYU pragma: export
		#include "arrow/compute/exec/exec_plan.h" // IWYU pragma: export
		#include "arrow/compute/exec/groupby.h" // IWYU pragma: export