[Exec](join) Support column string64 to avoid join failed in string size overflow the uint32 (#33511) #33850
Conversation
…ize overflow the uint32 (apache#33511)
|
Thank you for your contribution to Apache Doris. Since 2024-03-18, the Document has been moved to doris-website. |
![github-actions[bot]](https://avatars.githubusercontent.com/in/15368?s=60&v=4){kind=small}
| for (size_t i = 0; i < length; ++i) | ||
| cur_offsets[old_size + i] = src_offsets[start + i] - nested_offset + prev_max_offset; |
There was a problem hiding this comment.
warning: statement should be inside braces [readability-braces-around-statements]
| for (size_t i = 0; i < length; ++i) | |
| cur_offsets[old_size + i] = src_offsets[start + i] - nested_offset + prev_max_offset; | |
| for (size_t i = 0; i < length; ++i) { | |
| cur_offsets[old_size + i] = src_offsets[start + i] - nested_offset + prev_max_offset; | |
| } |
| } | ||
| } | ||
|
|
||
| void ColumnMap::insert_range_from_ignore_overflow(const IColumn& src, size_t start, size_t length) { |
There was a problem hiding this comment.
warning: method 'insert_range_from_ignore_overflow' can be made static [readability-convert-member-functions-to-static]
be/src/vec/columns/column_map.h:104:
- void insert_range_from_ignore_overflow(const IColumn& src, size_t start,
+ static void insert_range_from_ignore_overflow(const IColumn& src, size_t start,| } | ||
| } | ||
|
|
||
| void ColumnStruct::insert_range_from_ignore_overflow(const IColumn& src, size_t start, |
There was a problem hiding this comment.
warning: method 'insert_range_from_ignore_overflow' can be made static [readability-convert-member-functions-to-static]
be/src/vec/columns/column_struct.h:147:
- void insert_range_from_ignore_overflow(const IColumn& src, size_t start,
+ static void insert_range_from_ignore_overflow(const IColumn& src, size_t start,| void sort_column(const ColumnString64& column, EqualFlags& flags, IColumn::Permutation& perms, | ||
| EqualRange& range, bool last_column) const { |
There was a problem hiding this comment.
warning: method 'sort_column' can be made static [readability-convert-member-functions-to-static]
| void sort_column(const ColumnString64& column, EqualFlags& flags, IColumn::Permutation& perms, | |
| EqualRange& range, bool last_column) const { | |
| static void sort_column(const ColumnString64& column, EqualFlags& flags, IColumn::Permutation& perms, | |
| EqualRange& range, bool last_column) { |
3 participants
Proposed changes
Issue Number: close #xxx
Further comments
If this is a relatively large or complex change, kick off the discussion at dev@doris.apache.org by explaining why you chose the solution you did and what alternatives you considered, etc...