This is an automated email from the ASF dual-hosted git repository.
yiguolei pushed a commit to branch branch-2.1
in repository https://gitbox.apache.org/repos/asf/doris.git
The following commit(s) were added to refs/heads/branch-2.1 by this push:
new 088edffded5 [fix](column) remove string64 in check_chars_length
(#49537)
088edffded5 is described below
commit 088edffded5fb74a14d8e2ea0a388c17fc191733
Author: Mryange <yanxuech...@selectdb.com>
AuthorDate: Thu Mar 27 19:47:58 2025 +0800
[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 b9d684eb53a..3c912043f1c 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<T, UInt32>) {
+ 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);
+ }
}
}
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscr...@doris.apache.org
For additional commands, e-mail: commits-h...@doris.apache.org
