zclllyybb commented on code in PR #57828:
URL: https://github.com/apache/doris/pull/57828#discussion_r2560192297
##########
be/src/vec/exprs/lambda_function/varray_sort_function.cpp:
##########
@@ -81,18 +81,18 @@ class ArraySortFunction : public LambdaFunction {
auto column = column_ptr->convert_to_full_column_if_const();
auto input_rows = column->size();
- auto outside_null_map = ColumnUInt8::create(input_rows, 0);
auto arg_type = type_ptr;
auto arg_column = column;
+ ColumnPtr outside_null_map = nullptr;
+
if (arg_column->is_nullable()) {
arg_column = assert_cast<const
ColumnNullable*>(column.get())->get_nested_column_ptr();
- const auto& column_array_nullmap =
- assert_cast<const
ColumnNullable*>(column.get())->get_null_map_column();
+ outside_null_map = assert_cast<const ColumnNullable*>(column.get())
+ ->get_null_map_column_ptr()
+ ->assume_mutable();
Review Comment:
变成mutable之后又赋值给ColumnPtr，没意义吧？
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
