gianm commented on issue #5897: URL: https://github.com/apache/druid/issues/5897#issuecomment-1813790430
Thinking about this again, I am actually more excited about option 2 than I was in the past, especially now that we have proper arrays. It seems reasonable to me to say that empty MVDs should be treated by all grouping engines as if they were a null value (as `groupBy` currently does but `topN` currently does not). This avoids the implicit implosion which I do think will be confusing for people, and achieves equal behavior across the grouping engines. And proper array types provide ways for people to get other behaviors if they desire them. -- 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]
