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

…s negative, it will miss check test result apache#25734 (apache#25734) (apache#26551)
gnehil · Dec 4, 2023 · a745f7f · a745f7f
1 parent 12c7b71
commit a745f7f
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
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) {