walterddr commented on issue #12304: URL: https://github.com/apache/pinot/issues/12304#issuecomment-1925435872
this will work only in happy-path. it is not guarantee that the validated top is a always SqlSelect for normal queries. but it is a good starting point. ( for example it could be a SqlSetOperator if the top is a UNION ) i would suggest try out a whole bunch of queries and see if there's any query patterns that could evade these assumption (those could be a good test cases to be added) -- 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-unsubscr...@pinot.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: commits-unsubscr...@pinot.apache.org For additional commands, e-mail: commits-h...@pinot.apache.org