database/doris
2
0
Fork 0
Code Issues Pull Requests Actions 3 Packages Projects Releases Wiki Activity

[Fix](thrift api) column should be converted if const before serialize to arrow format (#26111)

Browse Source 
```
select null;
```

The above sql from thrift api will cause crash in `write_column_to_arrow` since it's column const,
we should convert to full column
This commit is contained in:
lihangyu
2023-10-30 19:56:19 +08:00
committed by GitHub
parent 2a74d9a8c8
commit 619f2bbbda
1 changed files with 2 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

3
be/src/util/arrow/block_convertor.cpp
View File

@ -388,8 +388,9 @@ Status FromBlockConverter::convert(std::shared_ptr<arrow::RecordBatch>* out) {
return to_doris_status(arrow_st);
}
_cur_builder = builder.get();
auto column = _cur_col->convert_to_full_column_if_const();
try {
_cur_type->get_serde()->write_column_to_arrow(*_cur_col, nullptr, _cur_builder,
_cur_type->get_serde()->write_column_to_arrow(*column, nullptr, _cur_builder,
_cur_start, _cur_start + _cur_rows);
} catch (std::exception& e) {
return Status::InternalError("Fail to convert block data to arrow data, error: {}",