Velox shows different behavior compared to Presto on non boolean aggregation masks

[kgpai]

Bug description

Velox shows different behavior compared to Presto java

SELECT sum(c0) FILTER(WHERE c0), count(*) FILTER(WHERE c0), c1 from (values (CAST(10 AS BIGINT), 'a')) t(c0, c1) GROUP BY c1;

 NULL  |     0 | a

Corresponding Query in Presto Java:

SELECT sum(c0) FILTER(WHERE c0), count(*) FILTER(WHERE c0), c1 from (values (CAST(10 AS BIGINT), 'a')) t(c0, c1) GROUP BY c1;

    10 |     1 | a

Whether the behavior is correct on Presto or Velox end is yet to be determined. Creating this issue to track this discrepancy.

System information

N/A

Relevant logs

N/A

[Yuhta]

Note that Presto Java does not allow non-boolean filter expression in non-aggregate filters. The implicit cast looks unsafe for a high level language like SQL.