github-actions[bot] commented on code in PR #42909:
URL: https://github.com/apache/doris/pull/42909#discussion_r1823678562


##########
be/src/pipeline/exec/exchange_sink_operator.cpp:
##########
@@ -389,7 +382,6 @@ void 
ExchangeSinkOperatorX::_handle_eof_channel(RuntimeState* state, ChannelPtrT
 Status ExchangeSinkOperatorX::sink(RuntimeState* state, vectorized::Block* 
block, bool eos) {

Review Comment:
   warning: function 'sink' has cognitive complexity of 237 (threshold 50) 
[readability-function-cognitive-complexity]
   ```cpp
   Status ExchangeSinkOperatorX::sink(RuntimeState* state, vectorized::Block* 
block, bool eos) {
                                 ^
   ```
   <details>
   <summary>Additional context</summary>
   
   **be/src/pipeline/exec/exchange_sink_operator.cpp:392:** +1, including 
nesting penalty of 0, nesting level increased to 1
   ```cpp
       if (all_receiver_eof) {
       ^
   ```
   **be/src/pipeline/exec/exchange_sink_operator.cpp:400:** +1, including 
nesting penalty of 0, nesting level increased to 1
   ```cpp
       if (_part_type == TPartitionType::UNPARTITIONED || 
local_state.channels.size() == 1) {
       ^
   ```
   **be/src/pipeline/exec/exchange_sink_operator.cpp:404:** +2, including 
nesting penalty of 1, nesting level increased to 2
   ```cpp
           if (local_state.only_local_exchange) {
           ^
   ```
   **be/src/pipeline/exec/exchange_sink_operator.cpp:405:** +3, including 
nesting penalty of 2, nesting level increased to 3
   ```cpp
               if (!block->empty()) {
               ^
   ```
   **be/src/pipeline/exec/exchange_sink_operator.cpp:420:** +1, nesting level 
increased to 2
   ```cpp
           } else {
             ^
   ```
   **be/src/pipeline/exec/exchange_sink_operator.cpp:424:** +3, including 
nesting penalty of 2, nesting level increased to 3
   ```cpp
                   
RETURN_IF_ERROR(local_state._serializer.next_serialized_block(
                   ^
   ```
   **be/src/common/status.h:629:** expanded from macro 'RETURN_IF_ERROR'
   ```cpp
       do {                                \
       ^
   ```
   **be/src/pipeline/exec/exchange_sink_operator.cpp:424:** +4, including 
nesting penalty of 3, nesting level increased to 4
   ```cpp
                   
RETURN_IF_ERROR(local_state._serializer.next_serialized_block(
                   ^
   ```
   **be/src/common/status.h:631:** expanded from macro 'RETURN_IF_ERROR'
   ```cpp
           if (UNLIKELY(!_status_.ok())) { \
           ^
   ```
   **be/src/pipeline/exec/exchange_sink_operator.cpp:427:** +3, including 
nesting penalty of 2, nesting level increased to 3
   ```cpp
                   if (serialized) {
                   ^
   ```
   **be/src/pipeline/exec/exchange_sink_operator.cpp:429:** +4, including 
nesting penalty of 3, nesting level increased to 4
   ```cpp
                       if (!cur_block.empty()) {
                       ^
   ```
   **be/src/pipeline/exec/exchange_sink_operator.cpp:430:** +1
   ```cpp
                           DCHECK(eos || local_state._serializer.is_local()) << 
debug_string(state, 0);
                                      ^
   ```
   **be/src/pipeline/exec/exchange_sink_operator.cpp:431:** +5, including 
nesting penalty of 4, nesting level increased to 5
   ```cpp
                           
RETURN_IF_ERROR(local_state._serializer.serialize_block(
                           ^
   ```
   **be/src/common/status.h:629:** expanded from macro 'RETURN_IF_ERROR'
   ```cpp
       do {                                \
       ^
   ```
   **be/src/pipeline/exec/exchange_sink_operator.cpp:431:** +6, including 
nesting penalty of 5, nesting level increased to 6
   ```cpp
                           
RETURN_IF_ERROR(local_state._serializer.serialize_block(
                           ^
   ```
   **be/src/common/status.h:631:** expanded from macro 'RETURN_IF_ERROR'
   ```cpp
           if (UNLIKELY(!_status_.ok())) { \
           ^
   ```
   **be/src/pipeline/exec/exchange_sink_operator.cpp:434:** +1, nesting level 
increased to 4
   ```cpp
                       } else {
                         ^
   ```
   **be/src/pipeline/exec/exchange_sink_operator.cpp:460:** +4, including 
nesting penalty of 3, nesting level increased to 4
   ```cpp
                       if (moved) {
                       ^
   ```
   **be/src/pipeline/exec/exchange_sink_operator.cpp:462:** +1, nesting level 
increased to 4
   ```cpp
                       } else {
                         ^
   ```
   **be/src/pipeline/exec/exchange_sink_operator.cpp:470:** +1, nesting level 
increased to 1
   ```cpp
       } else if (_part_type == TPartitionType::RANDOM) {
              ^
   ```
   **be/src/pipeline/exec/exchange_sink_operator.cpp:473:** +2, including 
nesting penalty of 1, nesting level increased to 2
   ```cpp
           if (!current_channel->is_receiver_eof()) {
           ^
   ```
   **be/src/pipeline/exec/exchange_sink_operator.cpp:475:** +3, including 
nesting penalty of 2, nesting level increased to 3
   ```cpp
               if (current_channel->is_local()) {
               ^
   ```
   **be/src/pipeline/exec/exchange_sink_operator.cpp:477:** +4, including 
nesting penalty of 3, nesting level increased to 4
   ```cpp
                   HANDLE_CHANNEL_STATUS(state, current_channel, status);
                   ^
   ```
   **be/src/vec/sink/vdata_stream_sender.h:225:** expanded from macro 
'HANDLE_CHANNEL_STATUS'
   ```cpp
       do {                                                 \
       ^
   ```
   **be/src/pipeline/exec/exchange_sink_operator.cpp:477:** +5, including 
nesting penalty of 4, nesting level increased to 5
   ```cpp
                   HANDLE_CHANNEL_STATUS(state, current_channel, status);
                   ^
   ```
   **be/src/vec/sink/vdata_stream_sender.h:226:** expanded from macro 
'HANDLE_CHANNEL_STATUS'
   ```cpp
           if (status.is<ErrorCode::END_OF_FILE>()) {       \
           ^
   ```
   **be/src/pipeline/exec/exchange_sink_operator.cpp:477:** +1, nesting level 
increased to 5
   ```cpp
                   HANDLE_CHANNEL_STATUS(state, current_channel, status);
                   ^
   ```
   **be/src/vec/sink/vdata_stream_sender.h:228:** expanded from macro 
'HANDLE_CHANNEL_STATUS'
   ```cpp
           } else {                                         \
             ^
   ```
   **be/src/pipeline/exec/exchange_sink_operator.cpp:477:** +6, including 
nesting penalty of 5, nesting level increased to 6
   ```cpp
                   HANDLE_CHANNEL_STATUS(state, current_channel, status);
                   ^
   ```
   **be/src/vec/sink/vdata_stream_sender.h:229:** expanded from macro 
'HANDLE_CHANNEL_STATUS'
   ```cpp
               RETURN_IF_ERROR(status);                     \
               ^
   ```
   **be/src/common/status.h:629:** expanded from macro 'RETURN_IF_ERROR'
   ```cpp
       do {                                \
       ^
   ```
   **be/src/pipeline/exec/exchange_sink_operator.cpp:477:** +7, including 
nesting penalty of 6, nesting level increased to 7
   ```cpp
                   HANDLE_CHANNEL_STATUS(state, current_channel, status);
                   ^
   ```
   **be/src/vec/sink/vdata_stream_sender.h:229:** expanded from macro 
'HANDLE_CHANNEL_STATUS'
   ```cpp
               RETURN_IF_ERROR(status);                     \
               ^
   ```
   **be/src/common/status.h:631:** expanded from macro 'RETURN_IF_ERROR'
   ```cpp
           if (UNLIKELY(!_status_.ok())) { \
           ^
   ```
   **be/src/pipeline/exec/exchange_sink_operator.cpp:478:** +1, nesting level 
increased to 3
   ```cpp
               } else {
                 ^
   ```
   **be/src/pipeline/exec/exchange_sink_operator.cpp:480:** +4, including 
nesting penalty of 3, nesting level increased to 4
   ```cpp
                   
RETURN_IF_ERROR(local_state._serializer.serialize_block(block, pblock.get()));
                   ^
   ```
   **be/src/common/status.h:629:** expanded from macro 'RETURN_IF_ERROR'
   ```cpp
       do {                                \
       ^
   ```
   **be/src/pipeline/exec/exchange_sink_operator.cpp:480:** +5, including 
nesting penalty of 4, nesting level increased to 5
   ```cpp
                   
RETURN_IF_ERROR(local_state._serializer.serialize_block(block, pblock.get()));
                   ^
   ```
   **be/src/common/status.h:631:** expanded from macro 'RETURN_IF_ERROR'
   ```cpp
           if (UNLIKELY(!_status_.ok())) { \
           ^
   ```
   **be/src/pipeline/exec/exchange_sink_operator.cpp:482:** +4, including 
nesting penalty of 3, nesting level increased to 4
   ```cpp
                   HANDLE_CHANNEL_STATUS(state, current_channel, status);
                   ^
   ```
   **be/src/vec/sink/vdata_stream_sender.h:225:** expanded from macro 
'HANDLE_CHANNEL_STATUS'
   ```cpp
       do {                                                 \
       ^
   ```
   **be/src/pipeline/exec/exchange_sink_operator.cpp:482:** +5, including 
nesting penalty of 4, nesting level increased to 5
   ```cpp
                   HANDLE_CHANNEL_STATUS(state, current_channel, status);
                   ^
   ```
   **be/src/vec/sink/vdata_stream_sender.h:226:** expanded from macro 
'HANDLE_CHANNEL_STATUS'
   ```cpp
           if (status.is<ErrorCode::END_OF_FILE>()) {       \
           ^
   ```
   **be/src/pipeline/exec/exchange_sink_operator.cpp:482:** +1, nesting level 
increased to 5
   ```cpp
                   HANDLE_CHANNEL_STATUS(state, current_channel, status);
                   ^
   ```
   **be/src/vec/sink/vdata_stream_sender.h:228:** expanded from macro 
'HANDLE_CHANNEL_STATUS'
   ```cpp
           } else {                                         \
             ^
   ```
   **be/src/pipeline/exec/exchange_sink_operator.cpp:482:** +6, including 
nesting penalty of 5, nesting level increased to 6
   ```cpp
                   HANDLE_CHANNEL_STATUS(state, current_channel, status);
                   ^
   ```
   **be/src/vec/sink/vdata_stream_sender.h:229:** expanded from macro 
'HANDLE_CHANNEL_STATUS'
   ```cpp
               RETURN_IF_ERROR(status);                     \
               ^
   ```
   **be/src/common/status.h:629:** expanded from macro 'RETURN_IF_ERROR'
   ```cpp
       do {                                \
       ^
   ```
   **be/src/pipeline/exec/exchange_sink_operator.cpp:482:** +7, including 
nesting penalty of 6, nesting level increased to 7
   ```cpp
                   HANDLE_CHANNEL_STATUS(state, current_channel, status);
                   ^
   ```
   **be/src/vec/sink/vdata_stream_sender.h:229:** expanded from macro 
'HANDLE_CHANNEL_STATUS'
   ```cpp
               RETURN_IF_ERROR(status);                     \
               ^
   ```
   **be/src/common/status.h:631:** expanded from macro 'RETURN_IF_ERROR'
   ```cpp
           if (UNLIKELY(!_status_.ok())) { \
           ^
   ```
   **be/src/pipeline/exec/exchange_sink_operator.cpp:487:** +1, nesting level 
increased to 1
   ```cpp
       } else if (_part_type == TPartitionType::HASH_PARTITIONED ||
              ^
   ```
   **be/src/pipeline/exec/exchange_sink_operator.cpp:492:** +2, including 
nesting penalty of 1, nesting level increased to 2
   ```cpp
               RETURN_IF_ERROR(local_state._partitioner->do_partitioning(state, 
block));
               ^
   ```
   **be/src/common/status.h:629:** expanded from macro 'RETURN_IF_ERROR'
   ```cpp
       do {                                \
       ^
   ```
   **be/src/pipeline/exec/exchange_sink_operator.cpp:492:** +3, including 
nesting penalty of 2, nesting level increased to 3
   ```cpp
               RETURN_IF_ERROR(local_state._partitioner->do_partitioning(state, 
block));
               ^
   ```
   **be/src/common/status.h:631:** expanded from macro 'RETURN_IF_ERROR'
   ```cpp
           if (UNLIKELY(!_status_.ok())) { \
           ^
   ```
   **be/src/pipeline/exec/exchange_sink_operator.cpp:498:** +2, including 
nesting penalty of 1, nesting level increased to 2
   ```cpp
           if (_part_type == TPartitionType::HASH_PARTITIONED) {
           ^
   ```
   **be/src/pipeline/exec/exchange_sink_operator.cpp:500:** +3, including 
nesting penalty of 2, nesting level increased to 3
   ```cpp
               RETURN_IF_ERROR(channel_add_rows(
               ^
   ```
   **be/src/common/status.h:629:** expanded from macro 'RETURN_IF_ERROR'
   ```cpp
       do {                                \
       ^
   ```
   **be/src/pipeline/exec/exchange_sink_operator.cpp:500:** +4, including 
nesting penalty of 3, nesting level increased to 4
   ```cpp
               RETURN_IF_ERROR(channel_add_rows(
               ^
   ```
   **be/src/common/status.h:631:** expanded from macro 'RETURN_IF_ERROR'
   ```cpp
           if (UNLIKELY(!_status_.ok())) { \
           ^
   ```
   **be/src/pipeline/exec/exchange_sink_operator.cpp:503:** +1, nesting level 
increased to 2
   ```cpp
           } else {
             ^
   ```
   **be/src/pipeline/exec/exchange_sink_operator.cpp:505:** +3, including 
nesting penalty of 2, nesting level increased to 3
   ```cpp
               RETURN_IF_ERROR(channel_add_rows(
               ^
   ```
   **be/src/common/status.h:629:** expanded from macro 'RETURN_IF_ERROR'
   ```cpp
       do {                                \
       ^
   ```
   **be/src/pipeline/exec/exchange_sink_operator.cpp:505:** +4, including 
nesting penalty of 3, nesting level increased to 4
   ```cpp
               RETURN_IF_ERROR(channel_add_rows(
               ^
   ```
   **be/src/common/status.h:631:** expanded from macro 'RETURN_IF_ERROR'
   ```cpp
           if (UNLIKELY(!_status_.ok())) { \
           ^
   ```
   **be/src/pipeline/exec/exchange_sink_operator.cpp:517:** +1, nesting level 
increased to 1
   ```cpp
       } else if (_part_type == 
TPartitionType::TABLET_SINK_SHUFFLE_PARTITIONED) {
              ^
   ```
   **be/src/pipeline/exec/exchange_sink_operator.cpp:523:** +2, including 
nesting penalty of 1, nesting level increased to 2
   ```cpp
           RETURN_IF_ERROR(local_state._send_new_partition_batch());
           ^
   ```
   **be/src/common/status.h:629:** expanded from macro 'RETURN_IF_ERROR'
   ```cpp
       do {                                \
       ^
   ```
   **be/src/pipeline/exec/exchange_sink_operator.cpp:523:** +3, including 
nesting penalty of 2, nesting level increased to 3
   ```cpp
           RETURN_IF_ERROR(local_state._send_new_partition_batch());
           ^
   ```
   **be/src/common/status.h:631:** expanded from macro 'RETURN_IF_ERROR'
   ```cpp
           if (UNLIKELY(!_status_.ok())) { \
           ^
   ```
   **be/src/pipeline/exec/exchange_sink_operator.cpp:530:** +2, including 
nesting penalty of 1, nesting level increased to 2
   ```cpp
           if (input_rows > 0) {
           ^
   ```
   **be/src/pipeline/exec/exchange_sink_operator.cpp:535:** +3, including 
nesting penalty of 2, nesting level increased to 3
   ```cpp
               
RETURN_IF_ERROR(local_state._row_distribution.generate_rows_distribution(
               ^
   ```
   **be/src/common/status.h:629:** expanded from macro 'RETURN_IF_ERROR'
   ```cpp
       do {                                \
       ^
   ```
   **be/src/pipeline/exec/exchange_sink_operator.cpp:535:** +4, including 
nesting penalty of 3, nesting level increased to 4
   ```cpp
               
RETURN_IF_ERROR(local_state._row_distribution.generate_rows_distribution(
               ^
   ```
   **be/src/common/status.h:631:** expanded from macro 'RETURN_IF_ERROR'
   ```cpp
           if (UNLIKELY(!_status_.ok())) { \
           ^
   ```
   **be/src/pipeline/exec/exchange_sink_operator.cpp:541:** +3, including 
nesting penalty of 2, nesting level increased to 3
   ```cpp
               for (int idx = 0; idx < row_ids.size(); ++idx) {
               ^
   ```
   **be/src/pipeline/exec/exchange_sink_operator.cpp:549:** +2, including 
nesting penalty of 1, nesting level increased to 2
   ```cpp
           if (eos) {
           ^
   ```
   **be/src/pipeline/exec/exchange_sink_operator.cpp:551:** +3, including 
nesting penalty of 2, nesting level increased to 3
   ```cpp
               RETURN_IF_ERROR(local_state._send_new_partition_batch());
               ^
   ```
   **be/src/common/status.h:629:** expanded from macro 'RETURN_IF_ERROR'
   ```cpp
       do {                                \
       ^
   ```
   **be/src/pipeline/exec/exchange_sink_operator.cpp:551:** +4, including 
nesting penalty of 3, nesting level increased to 4
   ```cpp
               RETURN_IF_ERROR(local_state._send_new_partition_batch());
               ^
   ```
   **be/src/common/status.h:631:** expanded from macro 'RETURN_IF_ERROR'
   ```cpp
           if (UNLIKELY(!_status_.ok())) { \
           ^
   ```
   **be/src/pipeline/exec/exchange_sink_operator.cpp:557:** +2, including 
nesting penalty of 1, nesting level increased to 2
   ```cpp
               RETURN_IF_ERROR(channel_add_rows_with_idx(state, 
local_state.channels, num_channels,
               ^
   ```
   **be/src/common/status.h:629:** expanded from macro 'RETURN_IF_ERROR'
   ```cpp
       do {                                \
       ^
   ```
   **be/src/pipeline/exec/exchange_sink_operator.cpp:557:** +3, including 
nesting penalty of 2, nesting level increased to 3
   ```cpp
               RETURN_IF_ERROR(channel_add_rows_with_idx(state, 
local_state.channels, num_channels,
               ^
   ```
   **be/src/common/status.h:631:** expanded from macro 'RETURN_IF_ERROR'
   ```cpp
           if (UNLIKELY(!_status_.ok())) { \
           ^
   ```
   **be/src/pipeline/exec/exchange_sink_operator.cpp:568:** +1, nesting level 
increased to 1
   ```cpp
       } else if (_part_type == TPartitionType::TABLE_SINK_HASH_PARTITIONED) {
              ^
   ```
   **be/src/pipeline/exec/exchange_sink_operator.cpp:575:** +2, including 
nesting penalty of 1, nesting level increased to 2
   ```cpp
               RETURN_IF_ERROR(local_state._partitioner->do_partitioning(state, 
block));
               ^
   ```
   **be/src/common/status.h:629:** expanded from macro 'RETURN_IF_ERROR'
   ```cpp
       do {                                \
       ^
   ```
   **be/src/pipeline/exec/exchange_sink_operator.cpp:575:** +3, including 
nesting penalty of 2, nesting level increased to 3
   ```cpp
               RETURN_IF_ERROR(local_state._partitioner->do_partitioning(state, 
block));
               ^
   ```
   **be/src/common/status.h:631:** expanded from macro 'RETURN_IF_ERROR'
   ```cpp
           if (UNLIKELY(!_status_.ok())) { \
           ^
   ```
   **be/src/pipeline/exec/exchange_sink_operator.cpp:581:** +2, including 
nesting penalty of 1, nesting level increased to 2
   ```cpp
               RETURN_IF_ERROR(channel_add_rows_with_idx(state, 
local_state.channels,
               ^
   ```
   **be/src/common/status.h:629:** expanded from macro 'RETURN_IF_ERROR'
   ```cpp
       do {                                \
       ^
   ```
   **be/src/pipeline/exec/exchange_sink_operator.cpp:581:** +3, including 
nesting penalty of 2, nesting level increased to 3
   ```cpp
               RETURN_IF_ERROR(channel_add_rows_with_idx(state, 
local_state.channels,
               ^
   ```
   **be/src/common/status.h:631:** expanded from macro 'RETURN_IF_ERROR'
   ```cpp
           if (UNLIKELY(!_status_.ok())) { \
           ^
   ```
   **be/src/pipeline/exec/exchange_sink_operator.cpp:594:** +1, nesting level 
increased to 1
   ```cpp
       } else if (_part_type == TPartitionType::TABLE_SINK_RANDOM_PARTITIONED) {
              ^
   ```
   **be/src/pipeline/exec/exchange_sink_operator.cpp:598:** +2, including 
nesting penalty of 1, nesting level increased to 2
   ```cpp
           if (!current_channel->is_receiver_eof()) {
           ^
   ```
   **be/src/pipeline/exec/exchange_sink_operator.cpp:600:** +3, including 
nesting penalty of 2, nesting level increased to 3
   ```cpp
               if (current_channel->is_local()) {
               ^
   ```
   **be/src/pipeline/exec/exchange_sink_operator.cpp:602:** +4, including 
nesting penalty of 3, nesting level increased to 4
   ```cpp
                   HANDLE_CHANNEL_STATUS(state, current_channel, status);
                   ^
   ```
   **be/src/vec/sink/vdata_stream_sender.h:225:** expanded from macro 
'HANDLE_CHANNEL_STATUS'
   ```cpp
       do {                                                 \
       ^
   ```
   **be/src/pipeline/exec/exchange_sink_operator.cpp:602:** +5, including 
nesting penalty of 4, nesting level increased to 5
   ```cpp
                   HANDLE_CHANNEL_STATUS(state, current_channel, status);
                   ^
   ```
   **be/src/vec/sink/vdata_stream_sender.h:226:** expanded from macro 
'HANDLE_CHANNEL_STATUS'
   ```cpp
           if (status.is<ErrorCode::END_OF_FILE>()) {       \
           ^
   ```
   **be/src/pipeline/exec/exchange_sink_operator.cpp:602:** +1, nesting level 
increased to 5
   ```cpp
                   HANDLE_CHANNEL_STATUS(state, current_channel, status);
                   ^
   ```
   **be/src/vec/sink/vdata_stream_sender.h:228:** expanded from macro 
'HANDLE_CHANNEL_STATUS'
   ```cpp
           } else {                                         \
             ^
   ```
   **be/src/pipeline/exec/exchange_sink_operator.cpp:602:** +6, including 
nesting penalty of 5, nesting level increased to 6
   ```cpp
                   HANDLE_CHANNEL_STATUS(state, current_channel, status);
                   ^
   ```
   **be/src/vec/sink/vdata_stream_sender.h:229:** expanded from macro 
'HANDLE_CHANNEL_STATUS'
   ```cpp
               RETURN_IF_ERROR(status);                     \
               ^
   ```
   **be/src/common/status.h:629:** expanded from macro 'RETURN_IF_ERROR'
   ```cpp
       do {                                \
       ^
   ```
   **be/src/pipeline/exec/exchange_sink_operator.cpp:602:** +7, including 
nesting penalty of 6, nesting level increased to 7
   ```cpp
                   HANDLE_CHANNEL_STATUS(state, current_channel, status);
                   ^
   ```
   **be/src/vec/sink/vdata_stream_sender.h:229:** expanded from macro 
'HANDLE_CHANNEL_STATUS'
   ```cpp
               RETURN_IF_ERROR(status);                     \
               ^
   ```
   **be/src/common/status.h:631:** expanded from macro 'RETURN_IF_ERROR'
   ```cpp
           if (UNLIKELY(!_status_.ok())) { \
           ^
   ```
   **be/src/pipeline/exec/exchange_sink_operator.cpp:603:** +1, nesting level 
increased to 3
   ```cpp
               } else {
                 ^
   ```
   **be/src/pipeline/exec/exchange_sink_operator.cpp:605:** +4, including 
nesting penalty of 3, nesting level increased to 4
   ```cpp
                   
RETURN_IF_ERROR(local_state._serializer.serialize_block(block, pblock.get()));
                   ^
   ```
   **be/src/common/status.h:629:** expanded from macro 'RETURN_IF_ERROR'
   ```cpp
       do {                                \
       ^
   ```
   **be/src/pipeline/exec/exchange_sink_operator.cpp:605:** +5, including 
nesting penalty of 4, nesting level increased to 5
   ```cpp
                   
RETURN_IF_ERROR(local_state._serializer.serialize_block(block, pblock.get()));
                   ^
   ```
   **be/src/common/status.h:631:** expanded from macro 'RETURN_IF_ERROR'
   ```cpp
           if (UNLIKELY(!_status_.ok())) { \
           ^
   ```
   **be/src/pipeline/exec/exchange_sink_operator.cpp:607:** +4, including 
nesting penalty of 3, nesting level increased to 4
   ```cpp
                   HANDLE_CHANNEL_STATUS(state, current_channel, status);
                   ^
   ```
   **be/src/vec/sink/vdata_stream_sender.h:225:** expanded from macro 
'HANDLE_CHANNEL_STATUS'
   ```cpp
       do {                                                 \
       ^
   ```
   **be/src/pipeline/exec/exchange_sink_operator.cpp:607:** +5, including 
nesting penalty of 4, nesting level increased to 5
   ```cpp
                   HANDLE_CHANNEL_STATUS(state, current_channel, status);
                   ^
   ```
   **be/src/vec/sink/vdata_stream_sender.h:226:** expanded from macro 
'HANDLE_CHANNEL_STATUS'
   ```cpp
           if (status.is<ErrorCode::END_OF_FILE>()) {       \
           ^
   ```
   **be/src/pipeline/exec/exchange_sink_operator.cpp:607:** +1, nesting level 
increased to 5
   ```cpp
                   HANDLE_CHANNEL_STATUS(state, current_channel, status);
                   ^
   ```
   **be/src/vec/sink/vdata_stream_sender.h:228:** expanded from macro 
'HANDLE_CHANNEL_STATUS'
   ```cpp
           } else {                                         \
             ^
   ```
   **be/src/pipeline/exec/exchange_sink_operator.cpp:607:** +6, including 
nesting penalty of 5, nesting level increased to 6
   ```cpp
                   HANDLE_CHANNEL_STATUS(state, current_channel, status);
                   ^
   ```
   **be/src/vec/sink/vdata_stream_sender.h:229:** expanded from macro 
'HANDLE_CHANNEL_STATUS'
   ```cpp
               RETURN_IF_ERROR(status);                     \
               ^
   ```
   **be/src/common/status.h:629:** expanded from macro 'RETURN_IF_ERROR'
   ```cpp
       do {                                \
       ^
   ```
   **be/src/pipeline/exec/exchange_sink_operator.cpp:607:** +7, including 
nesting penalty of 6, nesting level increased to 7
   ```cpp
                   HANDLE_CHANNEL_STATUS(state, current_channel, status);
                   ^
   ```
   **be/src/vec/sink/vdata_stream_sender.h:229:** expanded from macro 
'HANDLE_CHANNEL_STATUS'
   ```cpp
               RETURN_IF_ERROR(status);                     \
               ^
   ```
   **be/src/common/status.h:631:** expanded from macro 'RETURN_IF_ERROR'
   ```cpp
           if (UNLIKELY(!_status_.ok())) { \
           ^
   ```
   **be/src/pipeline/exec/exchange_sink_operator.cpp:612:** +2, including 
nesting penalty of 1, nesting level increased to 2
   ```cpp
           if (_writer_count < local_state.channels.size()) {
           ^
   ```
   **be/src/pipeline/exec/exchange_sink_operator.cpp:613:** +3, including 
nesting penalty of 2, nesting level increased to 3
   ```cpp
               if (_data_processed >=
               ^
   ```
   **be/src/pipeline/exec/exchange_sink_operator.cpp:620:** +1, nesting level 
increased to 1
   ```cpp
       } else {
         ^
   ```
   **be/src/pipeline/exec/exchange_sink_operator.cpp:627:** +1, including 
nesting penalty of 0, nesting level increased to 1
   ```cpp
       if (eos) {
       ^
   ```
   
   </details>
   



-- 
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]

Reply via email to