somu-imply commented on code in PR #14510:
URL: https://github.com/apache/druid/pull/14510#discussion_r1286040275
##########
sql/src/test/java/org/apache/druid/sql/calcite/CalciteQueryTest.java:
##########
@@ -2408,7 +2408,8 @@ public void testExactCountDistinctWithFilter()
new CountAggregatorFactory("_a0"),
and(
not(selector("d0", null, null)),
- selector("a1", "0", null)
+ selector("a1", "0", null),
+ expressionFilter("\"d1\"")
Review Comment:
There has been a virtual column here
```
expressionVirtualColumn(
"v0",
NullHandling.replaceWithDefault()
? "(\"cnt\" == 1)"
: "((\"cnt\" == 1) > 0)",
ColumnType.LONG
)
```
which is being represented in planning by d1 and when there is an
aggregation on the `cnt` column it checks if that value is null or not. If you
think this is a bug, let me know
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
