refactor: tweak build_aggregate_partial (#17038)

* refactor: tweak build_aggregate_partial * tweak cargo audit
databendlabs · Dec 12, 2024 · a32b80f · a32b80f
1 parent 62abe30
commit a32b80f
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 1 deletion.
diff --git a/.cargo/audit.toml b/.cargo/audit.toml
@@ -51,4 +51,6 @@ ignore = [
     "RUSTSEC-2024-0376",
     #rustls network-reachable panic in `Acceptor::accept`
     "RUSTSEC-2024-0399",
+    # `idna` accepts Punycode labels that do not produce any non-ASCII when decoded
+    "RUSTSEC-2024-0421",
 ]
diff --git a/src/query/service/src/pipelines/builders/builder_aggregate.rs b/src/query/service/src/pipelines/builders/builder_aggregate.rs
@@ -129,7 +129,7 @@ impl PipelineBuilder {
         let method = DataBlock::choose_hash_method(&sample_block, group_cols, efficiently_memory)?;
 
         // Need a global atomic to read the max current radix bits hint
-        let partial_agg_config = if !self.is_exchange_neighbor {
+        let partial_agg_config = if self.ctx.get_cluster().is_empty() {
             HashTableConfig::default().with_partial(true, max_threads as usize)
         } else {
             HashTableConfig::default()