This is an automated email from the ASF dual-hosted git repository.
gortiz 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 cfc8063d9e1 Type bool_and/bool_or intermediate result as BOOLEAN for
consistency (#18657)
cfc8063d9e1 is described below
commit cfc8063d9e198bed73fa885f99767bd5b41ead51
Author: Gonzalo Ortiz Jaureguizar <[email protected]>
AuthorDate: Tue Jun 2 17:44:10 2026 +0200
Type bool_and/bool_or intermediate result as BOOLEAN for consistency
(#18657)
---
.../java/org/apache/pinot/segment/spi/AggregationFunctionType.java | 5 +++--
1 file changed, 3 insertions(+), 2 deletions(-)
diff --git
a/pinot-segment-spi/src/main/java/org/apache/pinot/segment/spi/AggregationFunctionType.java
b/pinot-segment-spi/src/main/java/org/apache/pinot/segment/spi/AggregationFunctionType.java
index 63937d082bf..dff3a10674c 100644
---
a/pinot-segment-spi/src/main/java/org/apache/pinot/segment/spi/AggregationFunctionType.java
+++
b/pinot-segment-spi/src/main/java/org/apache/pinot/segment/spi/AggregationFunctionType.java
@@ -168,8 +168,9 @@ public enum AggregationFunctionType {
STUNION("STUnion", ReturnTypes.VARBINARY, OperandTypes.BINARY,
SqlTypeName.OTHER),
// boolean aggregate functions
- BOOLAND("boolAnd", ReturnTypes.BOOLEAN, OperandTypes.BOOLEAN,
SqlTypeName.INTEGER),
- BOOLOR("boolOr", ReturnTypes.BOOLEAN, OperandTypes.BOOLEAN,
SqlTypeName.INTEGER),
+ // Intermediate result type is BOOLEAN, serialized through its stored type
(INT).
+ BOOLAND("boolAnd", ReturnTypes.BOOLEAN, OperandTypes.BOOLEAN,
SqlTypeName.BOOLEAN),
+ BOOLOR("boolOr", ReturnTypes.BOOLEAN, OperandTypes.BOOLEAN,
SqlTypeName.BOOLEAN),
// ExprMin and ExprMax
// TODO: revisit support for ExprMin/Max count in V2, particularly plug
query rewriter in the right place
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
