weixiangsun commented on pull request #8029: URL: https://github.com/apache/pinot/pull/8029#issuecomment-1060199775
> It might be good to have a two or three integration test level queries as well since both broker and server are involved. Otherwise looks good to me assuming that you take care of moving the syntax checks to Broker side. Do you have an example about how to add the integration test I can add? I do not see any query-related test against broker and server -- 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
