kishoreg commented on issue #7213: URL: https://github.com/apache/pinot/issues/7213#issuecomment-960422654
Thanks for putting this together. Very good write up. High level thoughts - can we do match recognize instead of window? It's much more modern and more powerful than window functions and is a superset - in the first version, let's make sure we only support functions where local aggregation is possible or the number of records returned by server is limited (this can be configured by the user).. without this, even simple window functions can bring down the server.. we will need this for joins and sub queries as well - should we create the query spi layer first to support FULL sql syntax parsing? This Will also allow us to build the window function operator incremental without invasive changes Overall, this is a great initiative and let's try to think few steps ahead and lay the right foundation -- 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]
