Introducing a new config to ignore nulls while computing String Cardinality (#12345)
authorsomu-imply <93540295+somu-imply@users.noreply.github.com>
Tue, 29 Mar 2022 21:31:36 +0000 (14:31 -0700)
committerGitHub <noreply@github.com>
Tue, 29 Mar 2022 21:31:36 +0000 (14:31 -0700)
commita1ea6581156bf415fe991cebf74182b24d5b2994
tree6fc29c68bf9880cc48049da2df16df466a57fe6e
parentf1841c644408fe5d3bf801e2ab695a75dc100da9
Introducing a new config to ignore nulls while computing String Cardinality (#12345)

* Counting nulls in String cardinality with a config

* Adding tests for the new config

* Wrapping the vectorize part to allow backward compatibility

* Adding different tests, cleaning the code and putting the check at the proper position, handling hasRow() and hasValue() changes

* Updating testcase and code

* Adding null handling test to improve coverage

* Checkstyle fix

* Adding 1 more change in docs

* Making docs clearer
core/src/main/java/org/apache/druid/common/config/NullHandling.java
core/src/main/java/org/apache/druid/common/config/NullValueHandlingConfig.java
core/src/test/java/org/apache/druid/common/config/NullHandlingTest.java
docs/configuration/index.md
processing/src/main/java/org/apache/druid/query/aggregation/cardinality/types/StringCardinalityAggregatorColumnSelectorStrategy.java
processing/src/main/java/org/apache/druid/query/aggregation/cardinality/vector/SingleValueStringCardinalityVectorProcessor.java
processing/src/test/java/org/apache/druid/query/aggregation/cardinality/CardinalityAggregatorTest.java
website/.spelling