From 297a5f27a5325140ac3bb4556ebe24018963392d Mon Sep 17 00:00:00 2001 From: lide Date: Thu, 14 Mar 2019 12:28:43 +0800 Subject: [PATCH] Add comment to avoid modification for variable_length (#750) --- be/src/exprs/expr.cpp | 5 +++++ 1 file changed, 5 insertions(+) diff --git a/be/src/exprs/expr.cpp b/be/src/exprs/expr.cpp index e971006751..7420f314f8 100644 --- a/be/src/exprs/expr.cpp +++ b/be/src/exprs/expr.cpp @@ -488,6 +488,11 @@ int Expr::compute_results_layout( data[i].variable_length = true; } else if (exprs[i]->type().type == TYPE_DECIMAL) { data[i].byte_size = get_byte_size(exprs[i]->type().type); + + // Although the current decimal has a fix-length, for the + // same value, it will work out different hash value due to the + // different memory represent if the variable_length here is set + // to false, so we have to keep it. data[i].variable_length = true; } else { data[i].byte_size = get_byte_size(exprs[i]->type().type);