yashmayya commented on PR #16990: URL: https://github.com/apache/pinot/pull/16990#issuecomment-3398370614
> Why did you decide to go this way? This was a regression in functionality from https://github.com/apache/pinot/pull/16570/, prior to which the UNION ALL operator was indeed capable of handling more than 2 children. In my opinion, the safest and quickest way forward to fix the regression was to return to the model that existed previously, instead of switching to a third different solution and risking further regressions. I'm open to discussing the pros and cons of this approach versus the rule based approach splitting up multi-child UNIONs into multiple binary UNIONs. -- 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]
