github-actions[bot] commented on code in PR #42574:
URL: https://github.com/apache/doris/pull/42574#discussion_r1818482536
##########
be/src/vec/sink/vtablet_block_convertor.cpp:
##########
@@ -186,11 +187,11 @@ DecimalType
OlapTableBlockConvertor::_get_decimalv3_min_or_max(const TypeDescrip
Status OlapTableBlockConvertor::_internal_validate_column(
Review Comment:
warning: function '_internal_validate_column' has cognitive complexity of
336 (threshold 50) [readability-function-cognitive-complexity]
```cpp
Status OlapTableBlockConvertor::_internal_validate_column(
^
```
<details>
<summary>Additional context</summary>
**be/src/vec/sink/vtablet_block_convertor.cpp:191:** +1
```cpp
DCHECK((rows == nullptr) || (rows->size() == row_count));
^
```
**be/src/vec/sink/vtablet_block_convertor.cpp:193:** nesting level increased
to 1
```cpp
auto set_invalid_and_append_error_msg = [&](size_t row) {
^
```
**be/src/vec/sink/vtablet_block_convertor.cpp:206:** +1, including nesting
penalty of 0, nesting level increased to 1
```cpp
auto& real_column_ptr = column_ptr == nullptr ? column :
(column_ptr->get_nested_column_ptr());
^
```
**be/src/vec/sink/vtablet_block_convertor.cpp:207:** +1, including nesting
penalty of 0, nesting level increased to 1
```cpp
auto null_map = column_ptr == nullptr ? nullptr :
column_ptr->get_null_map_data().data();
^
```
**be/src/vec/sink/vtablet_block_convertor.cpp:208:** nesting level increased
to 1
```cpp
auto need_to_validate = [&null_map, this](size_t j, size_t row) {
^
```
**be/src/vec/sink/vtablet_block_convertor.cpp:212:** nesting level increased
to 1
```cpp
auto string_column_checker = [&](const ColumnString* column_string) {
^
```
**be/src/vec/sink/vtablet_block_convertor.cpp:215:** +2, including nesting
penalty of 1, nesting level increased to 2
```cpp
if (type.len >= 0) {
^
```
**be/src/vec/sink/vtablet_block_convertor.cpp:221:** +2, including nesting
penalty of 1, nesting level increased to 2
```cpp
for (int64_t j = 0; j < row_count; ++j) {
^
```
**be/src/vec/sink/vtablet_block_convertor.cpp:225:** +2, including nesting
penalty of 1, nesting level increased to 2
```cpp
if (invalid_count) {
^
```
**be/src/vec/sink/vtablet_block_convertor.cpp:226:** +3, including nesting
penalty of 2, nesting level increased to 3
```cpp
for (size_t j = 0; j < row_count; ++j) {
^
```
**be/src/vec/sink/vtablet_block_convertor.cpp:227:** +4, including nesting
penalty of 3, nesting level increased to 4
```cpp
auto row = rows ? (*rows)[j] : j;
^
```
**be/src/vec/sink/vtablet_block_convertor.cpp:228:** +4, including nesting
penalty of 3, nesting level increased to 4
```cpp
if (need_to_validate(j, row)) {
^
```
**be/src/vec/sink/vtablet_block_convertor.cpp:231:** +5, including nesting
penalty of 4, nesting level increased to 5
```cpp
if (invalid) {
^
```
**be/src/vec/sink/vtablet_block_convertor.cpp:232:** +6, including nesting
penalty of 5, nesting level increased to 6
```cpp
if (str_val.size > type.len) {
^
```
**be/src/vec/sink/vtablet_block_convertor.cpp:239:** +1, nesting level
increased to 6
```cpp
} else if (str_val.size > limit) {
^
```
**be/src/vec/sink/vtablet_block_convertor.cpp:249:** +6, including nesting
penalty of 5, nesting level increased to 6
```cpp
RETURN_IF_ERROR(set_invalid_and_append_error_msg(row));
^
```
**be/src/common/status.h:629:** expanded from macro 'RETURN_IF_ERROR'
```cpp
do { \
^
```
**be/src/vec/sink/vtablet_block_convertor.cpp:249:** +7, including nesting
penalty of 6, nesting level increased to 7
```cpp
RETURN_IF_ERROR(set_invalid_and_append_error_msg(row));
^
```
**be/src/common/status.h:631:** expanded from macro 'RETURN_IF_ERROR'
```cpp
if (UNLIKELY(!_status_.ok())) { \
^
```
**be/src/vec/sink/vtablet_block_convertor.cpp:257:** +1, including nesting
penalty of 0, nesting level increased to 1
```cpp
switch (type.type) {
^
```
**be/src/vec/sink/vtablet_block_convertor.cpp:263:** +2, including nesting
penalty of 1, nesting level increased to 2
```cpp
RETURN_IF_ERROR(string_column_checker(column_string));
^
```
**be/src/common/status.h:629:** expanded from macro 'RETURN_IF_ERROR'
```cpp
do { \
^
```
**be/src/vec/sink/vtablet_block_convertor.cpp:263:** +3, including nesting
penalty of 2, nesting level increased to 3
```cpp
RETURN_IF_ERROR(string_column_checker(column_string));
^
```
**be/src/common/status.h:631:** expanded from macro 'RETURN_IF_ERROR'
```cpp
if (UNLIKELY(!_status_.ok())) { \
^
```
**be/src/vec/sink/vtablet_block_convertor.cpp:269:** +2, including nesting
penalty of 1, nesting level increased to 2
```cpp
for (size_t j = 0; j < row_count; ++j) {
^
```
**be/src/vec/sink/vtablet_block_convertor.cpp:270:** +3, including nesting
penalty of 2, nesting level increased to 3
```cpp
if (!_filter_map[j]) {
^
```
**be/src/vec/sink/vtablet_block_convertor.cpp:271:** +4, including nesting
penalty of 3, nesting level increased to 4
```cpp
if (is_nullable && column_ptr && column_ptr->is_null_at(j)) {
^
```
**be/src/vec/sink/vtablet_block_convertor.cpp:271:** +1
```cpp
if (is_nullable && column_ptr && column_ptr->is_null_at(j)) {
^
```
**be/src/vec/sink/vtablet_block_convertor.cpp:276:** +4, including nesting
penalty of 3, nesting level increased to 4
```cpp
if (invalid) {
^
```
**be/src/vec/sink/vtablet_block_convertor.cpp:279:** +5, including nesting
penalty of 4, nesting level increased to 5
```cpp
RETURN_IF_ERROR(set_invalid_and_append_error_msg(j));
^
```
**be/src/common/status.h:629:** expanded from macro 'RETURN_IF_ERROR'
```cpp
do { \
^
```
**be/src/vec/sink/vtablet_block_convertor.cpp:279:** +6, including nesting
penalty of 5, nesting level increased to 6
```cpp
RETURN_IF_ERROR(set_invalid_and_append_error_msg(j));
^
```
**be/src/common/status.h:631:** expanded from macro 'RETURN_IF_ERROR'
```cpp
if (UNLIKELY(!_status_.ok())) { \
^
```
**be/src/vec/sink/vtablet_block_convertor.cpp:291:** +2, including nesting
penalty of 1, nesting level increased to 2
```cpp
for (size_t j = 0; j < row_count; ++j) {
^
```
**be/src/vec/sink/vtablet_block_convertor.cpp:292:** +3, including nesting
penalty of 2, nesting level increased to 3
```cpp
auto row = rows ? (*rows)[j] : j;
^
```
**be/src/vec/sink/vtablet_block_convertor.cpp:293:** +3, including nesting
penalty of 2, nesting level increased to 3
```cpp
if (need_to_validate(j, row)) {
^
```
**be/src/vec/sink/vtablet_block_convertor.cpp:298:** +4, including nesting
penalty of 3, nesting level increased to 4
```cpp
if (dec_val.greater_than_scale(type.scale)) {
^
```
**be/src/vec/sink/vtablet_block_convertor.cpp:302:** +5, including nesting
penalty of 4, nesting level increased to 5
```cpp
if (code != E_DEC_OK) {
^
```
**be/src/vec/sink/vtablet_block_convertor.cpp:308:** +4, including nesting
penalty of 3, nesting level increased to 4
```cpp
if (dec_val > max_decimalv2 || dec_val < min_decimalv2) {
^
```
**be/src/vec/sink/vtablet_block_convertor.cpp:308:** +1
```cpp
if (dec_val > max_decimalv2 || dec_val < min_decimalv2) {
^
```
**be/src/vec/sink/vtablet_block_convertor.cpp:318:** +4, including nesting
penalty of 3, nesting level increased to 4
```cpp
if (invalid) {
^
```
**be/src/vec/sink/vtablet_block_convertor.cpp:319:** +5, including nesting
penalty of 4, nesting level increased to 5
```cpp
RETURN_IF_ERROR(set_invalid_and_append_error_msg(row));
^
```
**be/src/common/status.h:629:** expanded from macro 'RETURN_IF_ERROR'
```cpp
do { \
^
```
**be/src/vec/sink/vtablet_block_convertor.cpp:319:** +6, including nesting
penalty of 5, nesting level increased to 6
```cpp
RETURN_IF_ERROR(set_invalid_and_append_error_msg(row));
^
```
**be/src/common/status.h:631:** expanded from macro 'RETURN_IF_ERROR'
```cpp
if (UNLIKELY(!_status_.ok())) { \
^
```
**be/src/vec/sink/vtablet_block_convertor.cpp:357:** +2, including nesting
penalty of 1, nesting level increased to 2
```cpp
CHECK_VALIDATION_FOR_DECIMALV3(vectorized::Decimal32);
^
```
**be/src/vec/sink/vtablet_block_convertor.cpp:333:** expanded from macro
'CHECK_VALIDATION_FOR_DECIMALV3'
```cpp
for (int j = 0; j < row_count; ++j) {
\
^
```
**be/src/vec/sink/vtablet_block_convertor.cpp:357:** +1
```cpp
CHECK_VALIDATION_FOR_DECIMALV3(vectorized::Decimal32);
^
```
**be/src/vec/sink/vtablet_block_convertor.cpp:335:** expanded from macro
'CHECK_VALIDATION_FOR_DECIMALV3'
```cpp
invalid_count += dec_val > max_decimal || dec_val < min_decimal;
\
^
```
**be/src/vec/sink/vtablet_block_convertor.cpp:357:** +2, including nesting
penalty of 1, nesting level increased to 2
```cpp
CHECK_VALIDATION_FOR_DECIMALV3(vectorized::Decimal32);
^
```
**be/src/vec/sink/vtablet_block_convertor.cpp:337:** expanded from macro
'CHECK_VALIDATION_FOR_DECIMALV3'
```cpp
if (invalid_count) {
\
^
```
**be/src/vec/sink/vtablet_block_convertor.cpp:357:** +3, including nesting
penalty of 2, nesting level increased to 3
```cpp
CHECK_VALIDATION_FOR_DECIMALV3(vectorized::Decimal32);
^
```
**be/src/vec/sink/vtablet_block_convertor.cpp:338:** expanded from macro
'CHECK_VALIDATION_FOR_DECIMALV3'
```cpp
for (size_t j = 0; j < row_count; ++j) {
\
^
```
**be/src/vec/sink/vtablet_block_convertor.cpp:357:** +4, including nesting
penalty of 3, nesting level increased to 4
```cpp
CHECK_VALIDATION_FOR_DECIMALV3(vectorized::Decimal32);
^
```
**be/src/vec/sink/vtablet_block_convertor.cpp:339:** expanded from macro
'CHECK_VALIDATION_FOR_DECIMALV3'
```cpp
auto row = rows ? (*rows)[j] : j;
\
^
```
**be/src/vec/sink/vtablet_block_convertor.cpp:357:** +4, including nesting
penalty of 3, nesting level increased to 4
```cpp
CHECK_VALIDATION_FOR_DECIMALV3(vectorized::Decimal32);
^
```
**be/src/vec/sink/vtablet_block_convertor.cpp:340:** expanded from macro
'CHECK_VALIDATION_FOR_DECIMALV3'
```cpp
if (need_to_validate(j, row)) {
\
^
```
**be/src/vec/sink/vtablet_block_convertor.cpp:357:** +5, including nesting
penalty of 4, nesting level increased to 5
```cpp
CHECK_VALIDATION_FOR_DECIMALV3(vectorized::Decimal32);
^
```
**be/src/vec/sink/vtablet_block_convertor.cpp:343:** expanded from macro
'CHECK_VALIDATION_FOR_DECIMALV3'
```cpp
if (dec_val > max_decimal || dec_val < min_decimal) {
\
^
```
**be/src/vec/sink/vtablet_block_convertor.cpp:357:** +1
```cpp
CHECK_VALIDATION_FOR_DECIMALV3(vectorized::Decimal32);
^
```
**be/src/vec/sink/vtablet_block_convertor.cpp:343:** expanded from macro
'CHECK_VALIDATION_FOR_DECIMALV3'
```cpp
if (dec_val > max_decimal || dec_val < min_decimal) {
\
^
```
**be/src/vec/sink/vtablet_block_convertor.cpp:357:** +5, including nesting
penalty of 4, nesting level increased to 5
```cpp
CHECK_VALIDATION_FOR_DECIMALV3(vectorized::Decimal32);
^
```
**be/src/vec/sink/vtablet_block_convertor.cpp:351:** expanded from macro
'CHECK_VALIDATION_FOR_DECIMALV3'
```cpp
if (invalid) {
\
^
```
**be/src/vec/sink/vtablet_block_convertor.cpp:357:** +6, including nesting
penalty of 5, nesting level increased to 6
```cpp
CHECK_VALIDATION_FOR_DECIMALV3(vectorized::Decimal32);
^
```
**be/src/vec/sink/vtablet_block_convertor.cpp:352:** expanded from macro
'CHECK_VALIDATION_FOR_DECIMALV3'
```cpp
RETURN_IF_ERROR(set_invalid_and_append_error_msg(row));
\
^
```
**be/src/common/status.h:629:** expanded from macro 'RETURN_IF_ERROR'
```cpp
do { \
^
```
**be/src/vec/sink/vtablet_block_convertor.cpp:357:** +7, including nesting
penalty of 6, nesting level increased to 7
```cpp
CHECK_VALIDATION_FOR_DECIMALV3(vectorized::Decimal32);
^
```
**be/src/vec/sink/vtablet_block_convertor.cpp:352:** expanded from macro
'CHECK_VALIDATION_FOR_DECIMALV3'
```cpp
RETURN_IF_ERROR(set_invalid_and_append_error_msg(row));
\
^
```
**be/src/common/status.h:631:** expanded from macro 'RETURN_IF_ERROR'
```cpp
if (UNLIKELY(!_status_.ok())) { \
^
```
**be/src/vec/sink/vtablet_block_convertor.cpp:361:** +2, including nesting
penalty of 1, nesting level increased to 2
```cpp
CHECK_VALIDATION_FOR_DECIMALV3(vectorized::Decimal64);
^
```
**be/src/vec/sink/vtablet_block_convertor.cpp:333:** expanded from macro
'CHECK_VALIDATION_FOR_DECIMALV3'
```cpp
for (int j = 0; j < row_count; ++j) {
\
^
```
**be/src/vec/sink/vtablet_block_convertor.cpp:361:** +1
```cpp
CHECK_VALIDATION_FOR_DECIMALV3(vectorized::Decimal64);
^
```
**be/src/vec/sink/vtablet_block_convertor.cpp:335:** expanded from macro
'CHECK_VALIDATION_FOR_DECIMALV3'
```cpp
invalid_count += dec_val > max_decimal || dec_val < min_decimal;
\
^
```
**be/src/vec/sink/vtablet_block_convertor.cpp:361:** +2, including nesting
penalty of 1, nesting level increased to 2
```cpp
CHECK_VALIDATION_FOR_DECIMALV3(vectorized::Decimal64);
^
```
**be/src/vec/sink/vtablet_block_convertor.cpp:337:** expanded from macro
'CHECK_VALIDATION_FOR_DECIMALV3'
```cpp
if (invalid_count) {
\
^
```
**be/src/vec/sink/vtablet_block_convertor.cpp:361:** +3, including nesting
penalty of 2, nesting level increased to 3
```cpp
CHECK_VALIDATION_FOR_DECIMALV3(vectorized::Decimal64);
^
```
**be/src/vec/sink/vtablet_block_convertor.cpp:338:** expanded from macro
'CHECK_VALIDATION_FOR_DECIMALV3'
```cpp
for (size_t j = 0; j < row_count; ++j) {
\
^
```
**be/src/vec/sink/vtablet_block_convertor.cpp:361:** +4, including nesting
penalty of 3, nesting level increased to 4
```cpp
CHECK_VALIDATION_FOR_DECIMALV3(vectorized::Decimal64);
^
```
**be/src/vec/sink/vtablet_block_convertor.cpp:339:** expanded from macro
'CHECK_VALIDATION_FOR_DECIMALV3'
```cpp
auto row = rows ? (*rows)[j] : j;
\
^
```
**be/src/vec/sink/vtablet_block_convertor.cpp:361:** +4, including nesting
penalty of 3, nesting level increased to 4
```cpp
CHECK_VALIDATION_FOR_DECIMALV3(vectorized::Decimal64);
^
```
**be/src/vec/sink/vtablet_block_convertor.cpp:340:** expanded from macro
'CHECK_VALIDATION_FOR_DECIMALV3'
```cpp
if (need_to_validate(j, row)) {
\
^
```
**be/src/vec/sink/vtablet_block_convertor.cpp:361:** +5, including nesting
penalty of 4, nesting level increased to 5
```cpp
CHECK_VALIDATION_FOR_DECIMALV3(vectorized::Decimal64);
^
```
**be/src/vec/sink/vtablet_block_convertor.cpp:343:** expanded from macro
'CHECK_VALIDATION_FOR_DECIMALV3'
```cpp
if (dec_val > max_decimal || dec_val < min_decimal) {
\
^
```
**be/src/vec/sink/vtablet_block_convertor.cpp:361:** +1
```cpp
CHECK_VALIDATION_FOR_DECIMALV3(vectorized::Decimal64);
^
```
**be/src/vec/sink/vtablet_block_convertor.cpp:343:** expanded from macro
'CHECK_VALIDATION_FOR_DECIMALV3'
```cpp
if (dec_val > max_decimal || dec_val < min_decimal) {
\
^
```
**be/src/vec/sink/vtablet_block_convertor.cpp:361:** +5, including nesting
penalty of 4, nesting level increased to 5
```cpp
CHECK_VALIDATION_FOR_DECIMALV3(vectorized::Decimal64);
^
```
**be/src/vec/sink/vtablet_block_convertor.cpp:351:** expanded from macro
'CHECK_VALIDATION_FOR_DECIMALV3'
```cpp
if (invalid) {
\
^
```
**be/src/vec/sink/vtablet_block_convertor.cpp:361:** +6, including nesting
penalty of 5, nesting level increased to 6
```cpp
CHECK_VALIDATION_FOR_DECIMALV3(vectorized::Decimal64);
^
```
**be/src/vec/sink/vtablet_block_convertor.cpp:352:** expanded from macro
'CHECK_VALIDATION_FOR_DECIMALV3'
```cpp
RETURN_IF_ERROR(set_invalid_and_append_error_msg(row));
\
^
```
**be/src/common/status.h:629:** expanded from macro 'RETURN_IF_ERROR'
```cpp
do { \
^
```
**be/src/vec/sink/vtablet_block_convertor.cpp:361:** +7, including nesting
penalty of 6, nesting level increased to 7
```cpp
CHECK_VALIDATION_FOR_DECIMALV3(vectorized::Decimal64);
^
```
**be/src/vec/sink/vtablet_block_convertor.cpp:352:** expanded from macro
'CHECK_VALIDATION_FOR_DECIMALV3'
```cpp
RETURN_IF_ERROR(set_invalid_and_append_error_msg(row));
\
^
```
**be/src/common/status.h:631:** expanded from macro 'RETURN_IF_ERROR'
```cpp
if (UNLIKELY(!_status_.ok())) { \
^
```
**be/src/vec/sink/vtablet_block_convertor.cpp:365:** +2, including nesting
penalty of 1, nesting level increased to 2
```cpp
CHECK_VALIDATION_FOR_DECIMALV3(vectorized::Decimal128V3);
^
```
**be/src/vec/sink/vtablet_block_convertor.cpp:333:** expanded from macro
'CHECK_VALIDATION_FOR_DECIMALV3'
```cpp
for (int j = 0; j < row_count; ++j) {
\
^
```
**be/src/vec/sink/vtablet_block_convertor.cpp:365:** +1
```cpp
CHECK_VALIDATION_FOR_DECIMALV3(vectorized::Decimal128V3);
^
```
**be/src/vec/sink/vtablet_block_convertor.cpp:335:** expanded from macro
'CHECK_VALIDATION_FOR_DECIMALV3'
```cpp
invalid_count += dec_val > max_decimal || dec_val < min_decimal;
\
^
```
**be/src/vec/sink/vtablet_block_convertor.cpp:365:** +2, including nesting
penalty of 1, nesting level increased to 2
```cpp
CHECK_VALIDATION_FOR_DECIMALV3(vectorized::Decimal128V3);
^
```
**be/src/vec/sink/vtablet_block_convertor.cpp:337:** expanded from macro
'CHECK_VALIDATION_FOR_DECIMALV3'
```cpp
if (invalid_count) {
\
^
```
**be/src/vec/sink/vtablet_block_convertor.cpp:365:** +3, including nesting
penalty of 2, nesting level increased to 3
```cpp
CHECK_VALIDATION_FOR_DECIMALV3(vectorized::Decimal128V3);
^
```
**be/src/vec/sink/vtablet_block_convertor.cpp:338:** expanded from macro
'CHECK_VALIDATION_FOR_DECIMALV3'
```cpp
for (size_t j = 0; j < row_count; ++j) {
\
^
```
**be/src/vec/sink/vtablet_block_convertor.cpp:365:** +4, including nesting
penalty of 3, nesting level increased to 4
```cpp
CHECK_VALIDATION_FOR_DECIMALV3(vectorized::Decimal128V3);
^
```
**be/src/vec/sink/vtablet_block_convertor.cpp:339:** expanded from macro
'CHECK_VALIDATION_FOR_DECIMALV3'
```cpp
auto row = rows ? (*rows)[j] : j;
\
^
```
**be/src/vec/sink/vtablet_block_convertor.cpp:365:** +4, including nesting
penalty of 3, nesting level increased to 4
```cpp
CHECK_VALIDATION_FOR_DECIMALV3(vectorized::Decimal128V3);
^
```
**be/src/vec/sink/vtablet_block_convertor.cpp:340:** expanded from macro
'CHECK_VALIDATION_FOR_DECIMALV3'
```cpp
if (need_to_validate(j, row)) {
\
^
```
**be/src/vec/sink/vtablet_block_convertor.cpp:365:** +5, including nesting
penalty of 4, nesting level increased to 5
```cpp
CHECK_VALIDATION_FOR_DECIMALV3(vectorized::Decimal128V3);
^
```
**be/src/vec/sink/vtablet_block_convertor.cpp:343:** expanded from macro
'CHECK_VALIDATION_FOR_DECIMALV3'
```cpp
if (dec_val > max_decimal || dec_val < min_decimal) {
\
^
```
**be/src/vec/sink/vtablet_block_convertor.cpp:365:** +1
```cpp
CHECK_VALIDATION_FOR_DECIMALV3(vectorized::Decimal128V3);
^
```
**be/src/vec/sink/vtablet_block_convertor.cpp:343:** expanded from macro
'CHECK_VALIDATION_FOR_DECIMALV3'
```cpp
if (dec_val > max_decimal || dec_val < min_decimal) {
\
^
```
**be/src/vec/sink/vtablet_block_convertor.cpp:365:** +5, including nesting
penalty of 4, nesting level increased to 5
```cpp
CHECK_VALIDATION_FOR_DECIMALV3(vectorized::Decimal128V3);
^
```
**be/src/vec/sink/vtablet_block_convertor.cpp:351:** expanded from macro
'CHECK_VALIDATION_FOR_DECIMALV3'
```cpp
if (invalid) {
\
^
```
**be/src/vec/sink/vtablet_block_convertor.cpp:365:** +6, including nesting
penalty of 5, nesting level increased to 6
```cpp
CHECK_VALIDATION_FOR_DECIMALV3(vectorized::Decimal128V3);
^
```
**be/src/vec/sink/vtablet_block_convertor.cpp:352:** expanded from macro
'CHECK_VALIDATION_FOR_DECIMALV3'
```cpp
RETURN_IF_ERROR(set_invalid_and_append_error_msg(row));
\
^
```
**be/src/common/status.h:629:** expanded from macro 'RETURN_IF_ERROR'
```cpp
do { \
^
```
**be/src/vec/sink/vtablet_block_convertor.cpp:365:** +7, including nesting
penalty of 6, nesting level increased to 7
```cpp
CHECK_VALIDATION_FOR_DECIMALV3(vectorized::Decimal128V3);
^
```
**be/src/vec/sink/vtablet_block_convertor.cpp:352:** expanded from macro
'CHECK_VALIDATION_FOR_DECIMALV3'
```cpp
RETURN_IF_ERROR(set_invalid_and_append_error_msg(row));
\
^
```
**be/src/common/status.h:631:** expanded from macro 'RETURN_IF_ERROR'
```cpp
if (UNLIKELY(!_status_.ok())) { \
^
```
**be/src/vec/sink/vtablet_block_convertor.cpp:369:** +2, including nesting
penalty of 1, nesting level increased to 2
```cpp
CHECK_VALIDATION_FOR_DECIMALV3(vectorized::Decimal256);
^
```
**be/src/vec/sink/vtablet_block_convertor.cpp:333:** expanded from macro
'CHECK_VALIDATION_FOR_DECIMALV3'
```cpp
for (int j = 0; j < row_count; ++j) {
\
^
```
**be/src/vec/sink/vtablet_block_convertor.cpp:369:** +1
```cpp
CHECK_VALIDATION_FOR_DECIMALV3(vectorized::Decimal256);
^
```
**be/src/vec/sink/vtablet_block_convertor.cpp:335:** expanded from macro
'CHECK_VALIDATION_FOR_DECIMALV3'
```cpp
invalid_count += dec_val > max_decimal || dec_val < min_decimal;
\
^
```
**be/src/vec/sink/vtablet_block_convertor.cpp:369:** +2, including nesting
penalty of 1, nesting level increased to 2
```cpp
CHECK_VALIDATION_FOR_DECIMALV3(vectorized::Decimal256);
^
```
**be/src/vec/sink/vtablet_block_convertor.cpp:337:** expanded from macro
'CHECK_VALIDATION_FOR_DECIMALV3'
```cpp
if (invalid_count) {
\
^
```
**be/src/vec/sink/vtablet_block_convertor.cpp:369:** +3, including nesting
penalty of 2, nesting level increased to 3
```cpp
CHECK_VALIDATION_FOR_DECIMALV3(vectorized::Decimal256);
^
```
**be/src/vec/sink/vtablet_block_convertor.cpp:338:** expanded from macro
'CHECK_VALIDATION_FOR_DECIMALV3'
```cpp
for (size_t j = 0; j < row_count; ++j) {
\
^
```
**be/src/vec/sink/vtablet_block_convertor.cpp:369:** +4, including nesting
penalty of 3, nesting level increased to 4
```cpp
CHECK_VALIDATION_FOR_DECIMALV3(vectorized::Decimal256);
^
```
**be/src/vec/sink/vtablet_block_convertor.cpp:339:** expanded from macro
'CHECK_VALIDATION_FOR_DECIMALV3'
```cpp
auto row = rows ? (*rows)[j] : j;
\
^
```
**be/src/vec/sink/vtablet_block_convertor.cpp:369:** +4, including nesting
penalty of 3, nesting level increased to 4
```cpp
CHECK_VALIDATION_FOR_DECIMALV3(vectorized::Decimal256);
^
```
**be/src/vec/sink/vtablet_block_convertor.cpp:340:** expanded from macro
'CHECK_VALIDATION_FOR_DECIMALV3'
```cpp
if (need_to_validate(j, row)) {
\
^
```
**be/src/vec/sink/vtablet_block_convertor.cpp:369:** +5, including nesting
penalty of 4, nesting level increased to 5
```cpp
CHECK_VALIDATION_FOR_DECIMALV3(vectorized::Decimal256);
^
```
**be/src/vec/sink/vtablet_block_convertor.cpp:343:** expanded from macro
'CHECK_VALIDATION_FOR_DECIMALV3'
```cpp
if (dec_val > max_decimal || dec_val < min_decimal) {
\
^
```
**be/src/vec/sink/vtablet_block_convertor.cpp:369:** +1
```cpp
CHECK_VALIDATION_FOR_DECIMALV3(vectorized::Decimal256);
^
```
**be/src/vec/sink/vtablet_block_convertor.cpp:343:** expanded from macro
'CHECK_VALIDATION_FOR_DECIMALV3'
```cpp
if (dec_val > max_decimal || dec_val < min_decimal) {
\
^
```
**be/src/vec/sink/vtablet_block_convertor.cpp:369:** +5, including nesting
penalty of 4, nesting level increased to 5
```cpp
CHECK_VALIDATION_FOR_DECIMALV3(vectorized::Decimal256);
^
```
**be/src/vec/sink/vtablet_block_convertor.cpp:351:** expanded from macro
'CHECK_VALIDATION_FOR_DECIMALV3'
```cpp
if (invalid) {
\
^
```
**be/src/vec/sink/vtablet_block_convertor.cpp:369:** +6, including nesting
penalty of 5, nesting level increased to 6
```cpp
CHECK_VALIDATION_FOR_DECIMALV3(vectorized::Decimal256);
^
```
**be/src/vec/sink/vtablet_block_convertor.cpp:352:** expanded from macro
'CHECK_VALIDATION_FOR_DECIMALV3'
```cpp
RETURN_IF_ERROR(set_invalid_and_append_error_msg(row));
\
^
```
**be/src/common/status.h:629:** expanded from macro 'RETURN_IF_ERROR'
```cpp
do { \
^
```
**be/src/vec/sink/vtablet_block_convertor.cpp:369:** +7, including nesting
penalty of 6, nesting level increased to 7
```cpp
CHECK_VALIDATION_FOR_DECIMALV3(vectorized::Decimal256);
^
```
**be/src/vec/sink/vtablet_block_convertor.cpp:352:** expanded from macro
'CHECK_VALIDATION_FOR_DECIMALV3'
```cpp
RETURN_IF_ERROR(set_invalid_and_append_error_msg(row));
\
^
```
**be/src/common/status.h:631:** expanded from macro 'RETURN_IF_ERROR'
```cpp
if (UNLIKELY(!_status_.ok())) { \
^
```
**be/src/vec/sink/vtablet_block_convertor.cpp:380:** +2, including nesting
penalty of 1, nesting level increased to 2
```cpp
for (size_t r = 0; r < row_count; ++r) {
^
```
**be/src/vec/sink/vtablet_block_convertor.cpp:381:** +3, including nesting
penalty of 2, nesting level increased to 3
```cpp
for (size_t c = offsets[r - 1]; c < offsets[r]; ++c) {
^
```
**be/src/vec/sink/vtablet_block_convertor.cpp:386:** +2, including nesting
penalty of 1, nesting level increased to 2
```cpp
RETURN_IF_ERROR(_validate_column(state, nested_type,
type.contains_nulls[0],
^
```
**be/src/common/status.h:629:** expanded from macro 'RETURN_IF_ERROR'
```cpp
do { \
^
```
**be/src/vec/sink/vtablet_block_convertor.cpp:386:** +3, including nesting
penalty of 2, nesting level increased to 3
```cpp
RETURN_IF_ERROR(_validate_column(state, nested_type,
type.contains_nulls[0],
^
```
**be/src/common/status.h:631:** expanded from macro 'RETURN_IF_ERROR'
```cpp
if (UNLIKELY(!_status_.ok())) { \
^
```
**be/src/vec/sink/vtablet_block_convertor.cpp:398:** +2, including nesting
penalty of 1, nesting level increased to 2
```cpp
for (size_t r = 0; r < row_count; ++r) {
^
```
**be/src/vec/sink/vtablet_block_convertor.cpp:399:** +3, including nesting
penalty of 2, nesting level increased to 3
```cpp
for (size_t c = offsets[r - 1]; c < offsets[r]; ++c) {
^
```
**be/src/vec/sink/vtablet_block_convertor.cpp:404:** +2, including nesting
penalty of 1, nesting level increased to 2
```cpp
RETURN_IF_ERROR(_validate_column(state, key_type,
type.contains_nulls[0],
^
```
**be/src/common/status.h:629:** expanded from macro 'RETURN_IF_ERROR'
```cpp
do { \
^
```
**be/src/vec/sink/vtablet_block_convertor.cpp:404:** +3, including nesting
penalty of 2, nesting level increased to 3
```cpp
RETURN_IF_ERROR(_validate_column(state, key_type,
type.contains_nulls[0],
^
```
**be/src/common/status.h:631:** expanded from macro 'RETURN_IF_ERROR'
```cpp
if (UNLIKELY(!_status_.ok())) { \
^
```
**be/src/vec/sink/vtablet_block_convertor.cpp:407:** +2, including nesting
penalty of 1, nesting level increased to 2
```cpp
RETURN_IF_ERROR(_validate_column(state, val_type,
type.contains_nulls[1],
^
```
**be/src/common/status.h:629:** expanded from macro 'RETURN_IF_ERROR'
```cpp
do { \
^
```
**be/src/vec/sink/vtablet_block_convertor.cpp:407:** +3, including nesting
penalty of 2, nesting level increased to 3
```cpp
RETURN_IF_ERROR(_validate_column(state, val_type,
type.contains_nulls[1],
^
```
**be/src/common/status.h:631:** expanded from macro 'RETURN_IF_ERROR'
```cpp
if (UNLIKELY(!_status_.ok())) { \
^
```
**be/src/vec/sink/vtablet_block_convertor.cpp:417:** +2, including nesting
penalty of 1, nesting level increased to 2
```cpp
for (size_t sc = 0; sc < column_struct->tuple_size(); ++sc) {
^
```
**be/src/vec/sink/vtablet_block_convertor.cpp:418:** +3, including nesting
penalty of 2, nesting level increased to 3
```cpp
RETURN_IF_ERROR(_validate_column(state, type.children[sc],
type.contains_nulls[sc],
^
```
**be/src/common/status.h:629:** expanded from macro 'RETURN_IF_ERROR'
```cpp
do { \
^
```
**be/src/vec/sink/vtablet_block_convertor.cpp:418:** +4, including nesting
penalty of 3, nesting level increased to 4
```cpp
RETURN_IF_ERROR(_validate_column(state, type.children[sc],
type.contains_nulls[sc],
^
```
**be/src/common/status.h:631:** expanded from macro 'RETURN_IF_ERROR'
```cpp
if (UNLIKELY(!_status_.ok())) { \
^
```
**be/src/vec/sink/vtablet_block_convertor.cpp:427:** +2, including nesting
penalty of 1, nesting level increased to 2
```cpp
if (column_string) {
^
```
**be/src/vec/sink/vtablet_block_convertor.cpp:428:** +3, including nesting
penalty of 2, nesting level increased to 3
```cpp
RETURN_IF_ERROR(string_column_checker(column_string));
^
```
**be/src/common/status.h:629:** expanded from macro 'RETURN_IF_ERROR'
```cpp
do { \
^
```
**be/src/vec/sink/vtablet_block_convertor.cpp:428:** +4, including nesting
penalty of 3, nesting level increased to 4
```cpp
RETURN_IF_ERROR(string_column_checker(column_string));
^
```
**be/src/common/status.h:631:** expanded from macro 'RETURN_IF_ERROR'
```cpp
if (UNLIKELY(!_status_.ok())) { \
^
```
**be/src/vec/sink/vtablet_block_convertor.cpp:440:** +1, including nesting
penalty of 0, nesting level increased to 1
```cpp
if ((!is_nullable || type == TYPE_OBJECT) && column_ptr) {
^
```
**be/src/vec/sink/vtablet_block_convertor.cpp:440:** +1
```cpp
if ((!is_nullable || type == TYPE_OBJECT) && column_ptr) {
^
```
**be/src/vec/sink/vtablet_block_convertor.cpp:440:** +1
```cpp
if ((!is_nullable || type == TYPE_OBJECT) && column_ptr) {
^
```
**be/src/vec/sink/vtablet_block_convertor.cpp:441:** +2, including nesting
penalty of 1, nesting level increased to 2
```cpp
for (int j = 0; j < row_count; ++j) {
^
```
**be/src/vec/sink/vtablet_block_convertor.cpp:442:** +3, including nesting
penalty of 2, nesting level increased to 3
```cpp
auto row = rows ? (*rows)[j] : j;
^
```
**be/src/vec/sink/vtablet_block_convertor.cpp:443:** +3, including nesting
penalty of 2, nesting level increased to 3
```cpp
if (null_map[j] && !_filter_map[row]) {
^
```
**be/src/vec/sink/vtablet_block_convertor.cpp:446:** +4, including nesting
penalty of 3, nesting level increased to 4
```cpp
RETURN_IF_ERROR(set_invalid_and_append_error_msg(row));
^
```
**be/src/common/status.h:629:** expanded from macro 'RETURN_IF_ERROR'
```cpp
do { \
^
```
**be/src/vec/sink/vtablet_block_convertor.cpp:446:** +5, including nesting
penalty of 4, nesting level increased to 5
```cpp
RETURN_IF_ERROR(set_invalid_and_append_error_msg(row));
^
```
**be/src/common/status.h:631:** expanded from macro 'RETURN_IF_ERROR'
```cpp
if (UNLIKELY(!_status_.ok())) { \
^
```
</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: 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
