You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@pinot.apache.org by ri...@apache.org on 2022/02/08 23:05:50 UTC

[pinot] branch master updated: fix aggregation error messages (#8165)

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

richardstartin pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/pinot.git


The following commit(s) were added to refs/heads/master by this push:
     new d479ebc  fix aggregation error messages (#8165)
d479ebc is described below

commit d479ebc87e1367ec3b7dbb1ac995230e6f1a4db2
Author: Richard Startin <ri...@startree.ai>
AuthorDate: Tue Feb 8 23:05:21 2022 +0000

    fix aggregation error messages (#8165)
---
 .../pinot/core/query/aggregation/function/MaxAggregationFunction.java   | 2 +-
 .../pinot/core/query/aggregation/function/SumAggregationFunction.java   | 2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/pinot-core/src/main/java/org/apache/pinot/core/query/aggregation/function/MaxAggregationFunction.java b/pinot-core/src/main/java/org/apache/pinot/core/query/aggregation/function/MaxAggregationFunction.java
index 55f65bd..ed80570 100644
--- a/pinot-core/src/main/java/org/apache/pinot/core/query/aggregation/function/MaxAggregationFunction.java
+++ b/pinot-core/src/main/java/org/apache/pinot/core/query/aggregation/function/MaxAggregationFunction.java
@@ -93,7 +93,7 @@ public class MaxAggregationFunction extends BaseSingleInputAggregationFunction<D
         break;
       }
       default:
-        throw new IllegalStateException("Cannot compute min for non-numeric type: " + blockValSet.getValueType());
+        throw new IllegalStateException("Cannot compute max for non-numeric type: " + blockValSet.getValueType());
     }
   }
 
diff --git a/pinot-core/src/main/java/org/apache/pinot/core/query/aggregation/function/SumAggregationFunction.java b/pinot-core/src/main/java/org/apache/pinot/core/query/aggregation/function/SumAggregationFunction.java
index bafe9e2..8b75642 100644
--- a/pinot-core/src/main/java/org/apache/pinot/core/query/aggregation/function/SumAggregationFunction.java
+++ b/pinot-core/src/main/java/org/apache/pinot/core/query/aggregation/function/SumAggregationFunction.java
@@ -86,7 +86,7 @@ public class SumAggregationFunction extends BaseSingleInputAggregationFunction<D
         break;
       }
       default:
-        throw new IllegalStateException("Cannot compute min for non-numeric type: " + blockValSet.getValueType());
+        throw new IllegalStateException("Cannot compute sum for non-numeric type: " + blockValSet.getValueType());
     }
     aggregationResultHolder.setValue(sum);
   }

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