> it might be much better to handle nullability centralized, not to require for 
> each implementer of AggregatorFactory to remember about nullability. E. g. 
> wrap with Comparator.nullsFirst(aggFactory.getComparator()) at the usage 
> sites of getComparator().

I like this idea. @nishantmonu51 any thoughts?

[ Full content available at: 
https://github.com/apache/incubator-druid/issues/5812 ]
This message was relayed via gitbox.apache.org for [email protected]

Reply via email to