[GitHub] [pinot] kishoreg commented on issue #7213: Support generic sliding window aggregations

[GitBox <gi...@apache.org>]

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: commits-unsubscribe@pinot.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@pinot.apache.org
For additional commands, e-mail: commits-help@pinot.apache.org