[pinot] branch master updated: don't toString every multivalue (#7649)

[ja...@apache.org]

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

jackie 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 41cdc02  don't toString every multivalue (#7649)
41cdc02 is described below

commit 41cdc02b27f69b5f6842e7530b4300f773e1b9de
Author: Richard Startin <ri...@startree.ai>
AuthorDate: Fri Oct 29 00:25:51 2021 +0100

    don't toString every multivalue (#7649)
---
 .../pinot/segment/local/recordtransformer/DataTypeTransformer.java  | 6 ++++--
 1 file changed, 4 insertions(+), 2 deletions(-)

diff --git a/pinot-segment-local/src/main/java/org/apache/pinot/segment/local/recordtransformer/DataTypeTransformer.java b/pinot-segment-local/src/main/java/org/apache/pinot/segment/local/recordtransformer/DataTypeTransformer.java
index fbf18bf..8586250 100644
--- a/pinot-segment-local/src/main/java/org/apache/pinot/segment/local/recordtransformer/DataTypeTransformer.java
+++ b/pinot-segment-local/src/main/java/org/apache/pinot/segment/local/recordtransformer/DataTypeTransformer.java
@@ -141,8 +141,10 @@ public class DataTypeTransformer implements RecordTransformer {
       if (numStandardizedValues == 1) {
         return standardizedValues.get(0);
       }
-      Preconditions.checkState(!isSingleValue, "Cannot read single-value from Object[]: %s for column: %s",
-          Arrays.toString(values), column);
+      if (isSingleValue) {
+        throw new IllegalArgumentException("Cannot read single-value from Object[]: " + Arrays.toString(values)
+            + " for column: " + column);
+      }
       return standardizedValues.toArray();
     }
     return value;

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