You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@druid.apache.org by GitBox <gi...@apache.org> on 2020/08/10 15:08:37 UTC
[GitHub] [druid] abhishekrb19 opened a new pull request #10260: Vectorization support for long, double, float min & max aggregators.
abhishekrb19 opened a new pull request #10260:
URL: https://github.com/apache/druid/pull/10260
As a follow up to https://github.com/apache/druid/pull/10211, this patch adds vectorization support for the following aggregators:
- `longMax`
- `doubleMin`
- `doubleMax`
- `floatMin`
- `floatMax`
Summary of changes:
* The vectorized implementations of the above min/max aggregators are each in *`VectorAggregator.java`
* Adds new unit test classes -- `FloatMinAggregationTest` and `FloatMaxAggregationTest` to test the floatMin and floatMax aggregations.
* Add new test cases and update existing tests in `CalciteQueryTest`, `TimeseriesQueryRunnerTest`, `GroupByQueryRunnerTest` and `GroupByTimeseriesQueryRunnerTest`.
* Update documentation to reflect that these aggregators now have vectorized versions.
/cc: @gianm
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@druid.apache.org
For additional commands, e-mail: commits-help@druid.apache.org
[GitHub] [druid] gianm merged pull request #10260: Vectorization support for long, double, float min & max aggregators.
Posted by GitBox <gi...@apache.org>.
gianm merged pull request #10260:
URL: https://github.com/apache/druid/pull/10260
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@druid.apache.org
For additional commands, e-mail: commits-help@druid.apache.org
[GitHub] [druid] gianm commented on pull request #10260: Vectorization support for long, double, float min & max aggregators.
Posted by GitBox <gi...@apache.org>.
gianm commented on pull request #10260:
URL: https://github.com/apache/druid/pull/10260#issuecomment-671618010
Thanks @abhishekrb19!
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@druid.apache.org
For additional commands, e-mail: commits-help@druid.apache.org