[GitHub] [pinot] somandal commented on pull request #10286: [multistage] Initial (phase 1) Query runtime for window functions - empty OVER() and OVER(PARTITION BY)

["somandal (via GitHub)" <gi...@apache.org>]

somandal commented on PR #10286:
URL: https://github.com/apache/pinot/pull/10286#issuecomment-1452962243

   > lgtm with some more comments. thank you @somandal for the contribution. this is a huge effort and we definitely appreciate the patience and consideration during the review/comment process.
   
   Thanks @walterddr! Appreciate the thorough review and the guidance on this from all involved.
   I've addressed all comments. PTAL when you get a chance


-- 
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