somandal commented on issue #11627: URL: https://github.com/apache/pinot/issues/11627#issuecomment-1726412553
One of the problems here is that the two window functions belong to different window groups. The default for ROW_NUMBER is ROWS type frame and for COUNT it’s RANGE type. I need to check if calcite splits the window groups for these if ORDER BY isn’t present, but my recent PR adds ROWS support for window functions with ORDER BY allowing such a query to run if for COUNT the explicit ROWS UNBOUNDED PRECEDING is passed. We decided not to support multiple window groups in the same query due to the complications around how exchanges should be done and which part should handle which window function I’ll get back to you later on my testing without ORDER BY, but just a caveat here that calcite doesn’t let you use ROWS without ORDER BY (it will compile but calcite will override frame type as RANGE) -- 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]
