amorynan opened a new pull request, #50032:
URL: https://github.com/apache/doris/pull/50032
### What problem does this PR solve?
somehow in sort situation we meet an empty block in some operator. so we
should deal with empty column in column_variant. otherwise we may meet core
like :
```
SUMMARY: UndefinedBehaviorSanitizer: undefined-behavior
/mnt/disk2/wangqiannan/amory/master/doris/be/src/vec/common/cow.h:198:12 in
*** Query id: f7c99aba0f494e70-8a36947b66383aa7 ***
*** is nereids: 1 ***
*** tablet id: 0 ***
*** Aborted at 1744620857 (unix time) try "date -d @1744620857" if you are
using GNU date ***
*** Current BE git commitID: 62a58bff4c ***
*** SIGSEGV address not mapped to object (@0x0) received by PID 483713 (TID
489737 OR 0x7fd98093e700) from PID 0; stack trace: ***
0# doris::signal::(anonymous namespace)::FailureSignalHandler(int,
siginfo_t*, void*) at
/mnt/disk2/wangqiannan/amory/master/doris/be/src/common/signal_handler.h:421
1# PosixSignals::chained_handler(int, siginfo*, void*) [clone .part.0] in
/mnt/disk2/wangqiannan/tool/jdk-17.0.10/lib/server/libjvm.so
2# JVM_handle_linux_signal in
/mnt/disk2/wangqiannan/tool/jdk-17.0.10/lib/server/libjvm.so
3# 0x00007FE4D07005B0 in /lib64/libc.so.6
4#
COW<doris::vectorized::IColumn>::intrusive_ptr<doris::vectorized::IColumn
const>::operator->() const at
/mnt/disk2/wangqiannan/amory/master/doris/be/src/vec/common/cow.h:198
5#
COW<doris::vectorized::IColumn>::chameleon_ptr<doris::vectorized::IColumn>::operator*()
at /mnt/disk2/wangqiannan/amory/master/doris/be/src/vec/common/cow.h:340
6# doris::vectorized::ColumnObject::Subcolumn::get_finalized_column() at
/mnt/disk2/wangqiannan/amory/master/doris/be/src/vec/columns/column_object.cpp:744
7# COW<doris::vectorized::IColumn>::mutable_ptr<doris::vectorized::IColumn>
doris::vectorized::ColumnObject::apply_for_subcolumns<doris::vectorized::ColumnObject::permute(doris::vectorized::PODArray<unsigned
long, 4096ul, Allocator<false, false, false, DefaultMemoryAllocator>, 16ul,
15ul> const&, unsigned long)
const::$_0>(doris::vectorized::ColumnObject::permute(doris::vectorized::PODArray<unsigned
long, 4096ul, Allocator<false, false, false, DefaultMemoryAllocator>, 16ul,
15ul> const&, unsigned long) const::$_0&&) const at
/mnt/disk2/wangqiannan/amory/master/doris/be/src/vec/columns/column_object.cpp:625
8#
doris::vectorized::ColumnObject::permute(doris::vectorized::PODArray<unsigned
long, 4096ul, Allocator<false, false, false, DefaultMemoryAllocator>, 16ul,
15ul> const&, unsigned long) const at
/mnt/disk2/wangqiannan/amory/master/doris/be/src/vec/columns/column_object.cpp:1125
9#
doris::vectorized::ColumnNullable::permute(doris::vectorized::PODArray<unsigned
long, 4096ul, Allocator<false, false, false, DefaultMemoryAllocator>, 16ul,
15ul> const&, unsigned long) const at
/mnt/disk2/wangqiannan/amory/master/doris/be/src/vec/columns/column_nullable.cpp:382
10# doris::vectorized::sort_block(doris::vectorized::Block&,
doris::vectorized::Block&,
std::vector<doris::vectorized::SortColumnDescription,
std::allocator<doris::vectorized::SortColumnDescription> > const&, unsigned
long) at
/mnt/disk2/wangqiannan/amory/master/doris/be/src/vec/core/sort_block.cpp:66
11# doris::vectorized::Sorter::partial_sort(doris::vectorized::Block&,
doris::vectorized::Block&) at
/mnt/disk2/wangqiannan/amory/master/doris/be/src/vec/common/sort/sorter.cpp:203
12#
doris::vectorized::PartitionSorter::append_block(doris::vectorized::Block*) at
/mnt/disk2/wangqiannan/amory/master/doris/be/src/vec/common/sort/partition_sorter.cpp:60
13# doris::pipeline::PartitionSortSinkOperatorX::sink(doris::RuntimeState*,
doris::vectorized::Block*, bool) at
/mnt/disk2/wangqiannan/amory/master/doris/be/src/pipeline/exec/partition_sort_sink_operator.cpp:215
14# doris::pipeline::PipelineTask::execute(bool*)::$_1::operator()() const
at
/mnt/disk2/wangqiannan/amory/master/doris/be/src/pipeline/pipeline_task.cpp:392
15# doris::pipeline::PipelineTask::execute(bool*) at
/mnt/disk2/wangqiannan/amory/master/doris/be/src/pipeline/pipeline_task.cpp:395
16# doris::pipeline::TaskScheduler::_do_work(unsigned long) at
/mnt/disk2/wangqiannan/amory/master/doris/be/src/pipeline/task_scheduler.cpp:138
17# doris::pipeline::TaskScheduler::start()::$_0::operator()() const at
/mnt/disk2/wangqiannan/amory/master/doris/be/src/pipeline/task_scheduler.cpp:64
18# void std::__invoke_impl<
```
Issue Number: close #xxx
Related PR: #xxx
Problem Summary:
### Release note
None
### Check List (For Author)
- Test <!-- At least one of them must be included. -->
- [ ] Regression test
- [ ] Unit Test
- [ ] Manual test (add detailed scripts or steps below)
- [ ] No need to test or manual test. Explain why:
- [ ] This is a refactor/code format and no logic has been changed.
- [ ] Previous test can cover this change.
- [ ] No code files have been changed.
- [ ] Other reason <!-- Add your reason? -->
- Behavior changed:
- [ ] No.
- [ ] Yes. <!-- Explain the behavior change -->
- Does this need documentation?
- [ ] No.
- [ ] Yes. <!-- Add document PR link here. eg:
https://github.com/apache/doris-website/pull/1214 -->
### Check List (For Reviewer who merge this PR)
- [ ] Confirm the release note
- [ ] Confirm test cases
- [ ] Confirm document
- [ ] Add branch pick label <!-- Add branch pick label that this PR should
merge into -->
--
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: commits-unsubscr...@doris.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscr...@doris.apache.org
For additional commands, e-mail: commits-h...@doris.apache.org
