walterddr opened a new pull request, #11255:
URL: https://github.com/apache/pinot/pull/11255
Summary
===
first draft to support histogram, supported syntax
```
select Dest, histogram(CAST(AirTime AS DOUBLE), 0, 100, 10) from
airlineStats group by Dest limit 1000
select Dest, histogram(CAST(AirTime AS DOUBLE), ARRAY[0, 10, 100]) from
airlineStats group by Dest limit 1000
```
Work Items
===
1. make ARRAY_VALUE_CONSTRUCTOR the default association with `ARRAY` keyword
(previously it was `ARRAY_QUERY_CONSTRUCTOR`)
2. Histogram can now register 2 different signatures --> (1) array-based and
(2) left, right, bin-size based
TODO
===
1. workaround for ArrayLiteral needs to be handled in `LiteralHintUtils`
- complication: RexExpressions.Literal now users FieldSpec.DataTye which
is not easy to represent array. maybe need to change to use
`DataSchema.ColumnDataType`
2. arrayValueConstructor needs to be a signature supported by leaf and
intermediate stage:
- for leaf, register a transform function type `arrayValueConststructor`
that returns identical array based on the literal
- for intermediate stage, register as a real-literal within the operator
so when reference to that row index (from the next operator) return that
directly.
CC @xiangfu0
--
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]
