[incubator-doris] branch master updated: [Improvement] reduce string size in serialization (#9550)

[mo...@apache.org]

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

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


The following commit(s) were added to refs/heads/master by this push:
     new 4312ef93d7 [Improvement] reduce string size in serialization (#9550)
4312ef93d7 is described below

commit 4312ef93d735dbf3ed5eb4f0be9d3bc16259115c
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 f2242615cf..c469d79190 100644
--- a/be/src/vec/columns/column_string.cpp
+++ b/be/src/vec/columns/column_string.cpp
@@ -164,7 +164,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;
@@ -178,7 +178,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