clintropolis commented on issue #8947: Unnecessary cartesian explosion if 
multi-value column is reused in expression
URL: 
https://github.com/apache/incubator-druid/issues/8947#issuecomment-558985943
 
 
   #8019 is the PR that changed the behavior to treat repeated uses of the same 
multi-value input as `cartesian_map` rather than `map`, though I can't find 
reason for why we decided to make this change. Based on this behavior 
definitely not making sense for conditional functions, it's probably worth 
revisiting and having a discussion if the `map` behavior was better, as it's 
relatively easy with just a few lines of code to change back (though a fair bit 
of tests to fix up).

----------------------------------------------------------------
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:
[email protected]


With regards,
Apache Git Services

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to