diff --git a/native-sql-engine/cpp/src/codegen/arrow_compute/ext/actions_impl.cc b/native-sql-engine/cpp/src/codegen/arrow_compute/ext/actions_impl.cc index 25e0fa67c..5a0860ac3 100644 --- a/native-sql-engine/cpp/src/codegen/arrow_compute/ext/actions_impl.cc +++ b/native-sql-engine/cpp/src/codegen/arrow_compute/ext/actions_impl.cc @@ -158,12 +158,9 @@ class UniqueAction : public ActionBase { if (!is_null) { cache_validity_[dest_group_id] = true; cache_[dest_group_id] = (CType)in_->GetView(row_id_); - //cache_.emplace(cache_.begin() + dest_group_id, in_->GetView(row_id_)); } else { cache_validity_[dest_group_id] = true; null_flag_[dest_group_id] = true; - // CType num; - // cache_.emplace(cache_.begin() + dest_group_id, num); } } row_id_++; diff --git a/native-sql-engine/cpp/src/codegen/arrow_compute/ext/hash_aggregate_kernel.cc b/native-sql-engine/cpp/src/codegen/arrow_compute/ext/hash_aggregate_kernel.cc index bfc552e9c..2288865e8 100644 --- a/native-sql-engine/cpp/src/codegen/arrow_compute/ext/hash_aggregate_kernel.cc +++ b/native-sql-engine/cpp/src/codegen/arrow_compute/ext/hash_aggregate_kernel.cc @@ -777,7 +777,6 @@ class HashAggregateKernel::Impl { } } else { for (int i = 0; i < length; i++) { - aggr_hash_table_->GetOrInsert( typed_key_in->GetView(i), [](int) {}, [](int) {}, &(indices[i])); } diff --git a/native-sql-engine/cpp/src/tests/test_utils.h b/native-sql-engine/cpp/src/tests/test_utils.h index 711b69dd4..d3afa459d 100644 --- a/native-sql-engine/cpp/src/tests/test_utils.h +++ b/native-sql-engine/cpp/src/tests/test_utils.h @@ -15,6 +15,8 @@ * limitations under the License. */ +#pragma once + #include #include #include