This is an automated email from the ASF dual-hosted git repository.
yiguolei pushed a commit to branch branch-1.1-lts
in repository https://gitbox.apache.org/repos/asf/doris.git
The following commit(s) were added to refs/heads/branch-1.1-lts by this push:
new 87ce8078fc [fix](join)ColumnNullable need handle const column with
nullable const value (#13866) (#13979)
87ce8078fc is described below
commit 87ce8078fcfdaed077d38f0115ef395db75d04f3
Author: starocean999 <[email protected]>
AuthorDate: Sat Nov 5 19:18:35 2022 +0800
[fix](join)ColumnNullable need handle const column with nullable const
value (#13866) (#13979)
---
be/src/vec/columns/column_nullable.cpp | 7 +++++++
1 file changed, 7 insertions(+)
diff --git a/be/src/vec/columns/column_nullable.cpp
b/be/src/vec/columns/column_nullable.cpp
index 41a58f063f..b9d4d1de1a 100644
--- a/be/src/vec/columns/column_nullable.cpp
+++ b/be/src/vec/columns/column_nullable.cpp
@@ -34,6 +34,13 @@ ColumnNullable::ColumnNullable(MutableColumnPtr&&
nested_column_, MutableColumnP
/// ColumnNullable cannot have constant nested column. But constant
argument could be passed. Materialize it.
nested_column = get_nested_column().convert_to_full_column_if_const();
+ // after convert const column to full column, it may be a nullable column
+ if (nested_column->is_nullable()) {
+ assert_cast<ColumnNullable&>(*nested_column).apply_null_map((const
ColumnUInt8&)*null_map);
+ null_map =
assert_cast<ColumnNullable&>(*nested_column).get_null_map_column_ptr();
+ nested_column =
assert_cast<ColumnNullable&>(*nested_column).get_nested_column_ptr();
+ }
+
if (!get_nested_column().can_be_inside_nullable()) {
LOG(FATAL) << get_nested_column().get_name() << " cannot be inside
Nullable column";
}
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
