Apache
/
druid
mirror of https://github.com/apache/druid.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

make single/multi value string column handling official in aggregation (#8428)

This commit is contained in:
Himanshu 2019-09-03 13:47:09 -07:00 committed by GitHub
parent 637a9e8614
commit ee4ebb496a
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
1 changed files with 6 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
processing/src/main/java/org/apache/druid/query/aggregation/AggregatorFactory.java
View File

@ -40,6 +40,12 @@ import java.util.Map;
* org.apache.druid.query.aggregation.cardinality.CardinalityAggregatorFactory}).
* Implementations of {@link AggregatorFactory} which need to Support Nullable Aggregations are encouraged
* to extend {@link NullableAggregatorFactory}.
*
* Implementations are also expected to correctly handle single/multi value string type columns as it makes sense
* for them e.g. doubleSum aggregator tries to parse the string value as double and assumes it to be zero if parsing
* fails.
* If it is a multi value column then each individual value should be taken into account for aggregation e.g. if a row
* had value ["1","1","1"] , doubleSum aggregation would take each of them and sum them to 3.
*/
@ExtensionPoint
public abstract class AggregatorFactory implements Cacheable