You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@doris.apache.org by mo...@apache.org on 2022/05/18 08:34:41 UTC

[incubator-doris] 06/09: [Improvement] reduce string size in serialization (#9550)

This is an automated email from the ASF dual-hosted git repository.

morningman pushed a commit to branch dev-1.0.1
in repository https://gitbox.apache.org/repos/asf/incubator-doris.git

commit f97b9d93871e26280138667dde29f1085730e836
Author: Gabriel <ga...@gmail.com>
AuthorDate: Tue May 17 22:38:34 2022 +0800

    [Improvement] reduce string size in serialization (#9550)
---
 be/src/vec/columns/column_string.cpp | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/be/src/vec/columns/column_string.cpp b/be/src/vec/columns/column_string.cpp
index 98ebf673b4..59f521b50c 100644
--- a/be/src/vec/columns/column_string.cpp
+++ b/be/src/vec/columns/column_string.cpp
@@ -163,7 +163,7 @@ ColumnPtr ColumnString::permute(const Permutation& perm, size_t limit) const {
 
 StringRef ColumnString::serialize_value_into_arena(size_t n, Arena& arena,
                                                    char const*& begin) const {
-    size_t string_size = size_at(n);
+    UInt32 string_size = size_at(n);
     size_t offset = offset_at(n);
 
     StringRef res;
@@ -177,7 +177,7 @@ StringRef ColumnString::serialize_value_into_arena(size_t n, Arena& arena,
 }
 
 const char* ColumnString::deserialize_and_insert_from_arena(const char* pos) {
-    const size_t string_size = unaligned_load<size_t>(pos);
+    const UInt32 string_size = unaligned_load<UInt32>(pos);
     pos += sizeof(string_size);
 
     const size_t old_size = chars.size();


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For additional commands, e-mail: commits-help@doris.apache.org