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

jlli pushed a commit to branch fix-array-agg-group-by
in repository https://gitbox.apache.org/repos/asf/pinot.git

commit 4254e0f9f15f9863ac8f605a172514686c859bee
Author: jlli_LinkedIn <j...@linkedin.com>
AuthorDate: Thu Apr 3 15:05:07 2025 -0700

    Fix Null value for group by queries when no aggregate was called
---
 .../pinot/core/operator/blocks/results/GroupByResultsBlock.java      | 5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)

diff --git 
a/pinot-core/src/main/java/org/apache/pinot/core/operator/blocks/results/GroupByResultsBlock.java
 
b/pinot-core/src/main/java/org/apache/pinot/core/operator/blocks/results/GroupByResultsBlock.java
index da09681d5d..425bb2052c 100644
--- 
a/pinot-core/src/main/java/org/apache/pinot/core/operator/blocks/results/GroupByResultsBlock.java
+++ 
b/pinot-core/src/main/java/org/apache/pinot/core/operator/blocks/results/GroupByResultsBlock.java
@@ -252,8 +252,9 @@ public class GroupByResultsBlock extends BaseResultsBlock {
         Object[] values = iterator.next().getValues();
         for (int i = 0; i < numColumns; i++) {
           Object value = values[i];
-          assert value != null;
-          if (storedColumnDataTypes[i] == ColumnDataType.OBJECT) {
+          if (value == null) {
+            dataTableBuilder.setNull(i);
+          } else if (storedColumnDataTypes[i] == ColumnDataType.OBJECT) {
             dataTableBuilder.setColumn(i, aggregationFunctions[i - 
numKeyColumns].serializeIntermediateResult(value));
           } else {
             setDataTableColumn(storedColumnDataTypes[i], dataTableBuilder, i, 
value);


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

Reply via email to