Skip to content

Commit

Permalink
chore: add err message to help capture new agg ht hang (databendlabs#…
Browse files Browse the repository at this point in the history
…15344)

* add err to capture new agg ht hang

* fix

* fix err info

---------

Co-authored-by: jw <[email protected]>
  • Loading branch information
Freejww and jw authored Apr 26, 2024
1 parent 8e67459 commit f2a49de
Showing 1 changed file with 11 additions and 0 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -19,6 +19,7 @@ use std::marker::PhantomData;
use std::sync::Arc;

use bumpalo::Bump;
use databend_common_exception::ErrorCode;
use databend_common_exception::Result;
use databend_common_expression::BlockMetaInfoDowncast;
use databend_common_expression::DataBlock;
Expand Down Expand Up @@ -196,6 +197,7 @@ impl<Method: HashMethodBounds, V: Copy + Send + Sync + 'static>
Ok(self.initialized_all_inputs)
}

#[allow(unused_assignments)]
fn add_bucket(&mut self, mut data_block: DataBlock) -> Result<(isize, usize)> {
let (mut bucket, mut partition_count) = (0, 0);
if let Some(block_meta) = data_block.get_meta() {
Expand Down Expand Up @@ -277,7 +279,16 @@ impl<Method: HashMethodBounds, V: Copy + Send + Sync + 'static>
(payload.bucket, payload.max_partition_count)
}
};
} else {
return Err(ErrorCode::Internal(format!(
"Internal, TransformPartitionBucket only recv AggregateMeta, but got {:?}",
block_meta
)));
}
} else {
return Err(ErrorCode::Internal(
"Internal, TransformPartitionBucket only recv DataBlock with meta.",
));
}

if self.all_inputs_init {
Expand Down

0 comments on commit f2a49de

Please sign in to comment.