Skip to content

Commit c807783

Browse files
committed
fix
1 parent 8875f53 commit c807783

File tree

3 files changed

+3
-15
lines changed

3 files changed

+3
-15
lines changed

src/query/service/src/pipelines/processors/transforms/recluster/range_bound_sampler.rs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -56,7 +56,7 @@ where
5656
pub fn add_block(&mut self, data: &DataBlock) {
5757
let rows = data.num_rows();
5858
assert!(rows > 0);
59-
let column = data.get_by_offset(self.offset).to_column(rows);
59+
let column = data.get_by_offset(self.offset).to_column();
6060

6161
let sample_size = std::cmp::min(self.sample_size, rows);
6262
let mut indices = (0..rows).collect::<Vec<_>>();

src/query/service/src/pipelines/processors/transforms/recluster/transform_add_order_column.rs

Lines changed: 1 addition & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -16,11 +16,9 @@ use std::marker::PhantomData;
1616
use std::sync::Arc;
1717

1818
use databend_common_exception::Result;
19-
use databend_common_expression::BlockEntry;
2019
use databend_common_expression::DataBlock;
2120
use databend_common_expression::DataSchemaRef;
2221
use databend_common_expression::SortColumnDescription;
23-
use databend_common_expression::Value;
2422
use databend_common_pipeline_transforms::sort::RowConverter;
2523
use databend_common_pipeline_transforms::sort::Rows;
2624
use databend_common_pipeline_transforms::Transform;
@@ -63,10 +61,7 @@ where
6361
.row_converter
6462
.convert(&order_by_cols, data.num_rows())?;
6563
let order_col = rows.to_column();
66-
data.add_column(BlockEntry {
67-
data_type: order_col.data_type(),
68-
value: Value::Column(order_col),
69-
});
64+
data.add_column(order_col);
7065
Ok(data)
7166
}
7267
}

src/query/service/src/pipelines/processors/transforms/recluster/transform_range_partition_indexer.rs

Lines changed: 1 addition & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -19,14 +19,10 @@ use std::time::Instant;
1919

2020
use databend_common_exception::Result;
2121
use databend_common_expression::types::ArgType;
22-
use databend_common_expression::types::DataType;
23-
use databend_common_expression::types::NumberDataType;
2422
use databend_common_expression::types::UInt64Type;
25-
use databend_common_expression::BlockEntry;
2623
use databend_common_expression::BlockMetaInfoDowncast;
2724
use databend_common_expression::DataBlock;
2825
use databend_common_expression::FromData;
29-
use databend_common_expression::Value;
3026
use databend_common_pipeline_core::processors::Event;
3127
use databend_common_pipeline_core::processors::InputPort;
3228
use databend_common_pipeline_core::processors::OutputPort;
@@ -164,10 +160,7 @@ where
164160
builder.push(low as u64);
165161
}
166162

167-
block.add_column(BlockEntry::new(
168-
DataType::Number(NumberDataType::UInt64),
169-
Value::Column(UInt64Type::from_data(builder)),
170-
));
163+
block.add_column(UInt64Type::from_data(builder));
171164
self.output_data.push_back(block);
172165
log::info!("Recluster range output: {:?}", start.elapsed());
173166
Ok(())

0 commit comments

Comments
 (0)