From 088edffded5fb74a14d8e2ea0a388c17fc191733 Mon Sep 17 00:00:00 2001 From: Mryange Date: Thu, 27 Mar 2025 19:47:58 +0800 Subject: [PATCH] [fix](column) remove string64 in check_chars_length (#49537) --- be/src/vec/columns/column_string.h | 14 ++++++++------ 1 file changed, 8 insertions(+), 6 deletions(-) diff --git a/be/src/vec/columns/column_string.h b/be/src/vec/columns/column_string.h index b9d684eb53..3c912043f1 100644 --- a/be/src/vec/columns/column_string.h +++ b/be/src/vec/columns/column_string.h @@ -61,12 +61,14 @@ public: static constexpr size_t MAX_STRINGS_OVERFLOW_SIZE = 128; void static check_chars_length(size_t total_length, size_t element_number) { - if (UNLIKELY(total_length > MAX_STRING_SIZE)) { - throw Exception( - ErrorCode::STRING_OVERFLOW_IN_VEC_ENGINE, - "string column length is too large: total_length={}, element_number={}, " - "you can set batch_size a number smaller than {} to avoid this error", - total_length, element_number, element_number); + if constexpr (std::is_same_v) { + if (UNLIKELY(total_length > MAX_STRING_SIZE)) { + throw Exception( + ErrorCode::STRING_OVERFLOW_IN_VEC_ENGINE, + "string column length is too large: total_length={}, element_number={}, " + "you can set batch_size a number smaller than {} to avoid this error", + total_length, element_number, element_number); + } } }