[fix](regression) fix regression framework bug: if real test result i…

…s negative, it will miss check test result (apache#25734)
XuJianxu · Dec 14, 2023 · 7c33ab0 · 7c33ab0
1 parent 4e26c43
commit 7c33ab0
Show file tree

Hide file tree

Showing 3 changed files with 4 additions and 3 deletions.
diff --git a/be/src/vec/functions/function_binary_arithmetic.h b/be/src/vec/functions/function_binary_arithmetic.h
@@ -79,7 +79,8 @@ struct OperationTraits {
  std::is_same_v<Op, DivideIntegralImpl<T, T>>;
  static constexpr bool can_overflow =
  (is_plus_minus || is_multiply) &&
- (IsDecimalV2<OpA> || IsDecimalV2<OpB> || IsDecimal256<OpA> || IsDecimal256<OpB>);
+ (IsDecimalV2<OpA> || IsDecimalV2<OpB> || IsDecimal128I<OpA> || IsDecimal128I<OpB> ||
+ IsDecimal256<OpA> || IsDecimal256<OpB>);
  static constexpr bool has_variadic_argument =
  !std::is_void_v<decltype(has_variadic_argument_types(std::declval<Op>()))>;
 };

diff --git a/regression-test/data/schema_change_p0/decimalv3/test_agg_keys_schema_change_decimalv3.out b/regression-test/data/schema_change_p0/decimalv3/test_agg_keys_schema_change_decimalv3.out
@@ -18,7 +18,7 @@
 0.11111111111111111111111111111111100000 11111111111111111111111111111.1100000000 0.11111111111111111111111111111111100000 11111111111111111111111111111.110
 
 -- !sql --
-0.11111111111111111111111111111111100000 9223372036854775.807 0.11111111111111111111111111111111100000 11111111111111111111111111111.110
+0.11111111111111111111111111111111100000 -999999999999999.999 0.11111111111111111111111111111111100000 11111111111111111111111111111.110
 
 -- !sql --
 0.11111111111111111111111111111111100000 -9223.372036854775808 0.11111111111111111111111111111111100000 11111111111111111111111111111.110

diff --git a/...ession-test/framework/src/main/groovy/org/apache/doris/regression/util/OutputUtils.groovy b/...ession-test/framework/src/main/groovy/org/apache/doris/regression/util/OutputUtils.groovy
@@ -76,7 +76,7 @@ class OutputUtils {
  double expectDouble = Double.parseDouble(expectCell)
  double realDouble = Double.parseDouble(realCell)
 
- double realRelativeError = Math.abs(expectDouble - realDouble) / realDouble
+ double realRelativeError = Math.abs(expectDouble - realDouble) / Math.abs(realDouble)
  double expectRelativeError = 1e-8
 
  if (expectRelativeError < realRelativeError) {