walterddr commented on PR #9653: URL: https://github.com/apache/pinot/pull/9653#issuecomment-1292619092
> LGTM! Change makes sense :) in case i missed something, this is just refactoring, correct? (there's no additional functionality in this PR) no functional changes. I am still trying to fully get rid of the last boilerplate code in the `ServerRequestUtils` and make it fully visitor pattern this is needed for doing dynamic filter poc - so that we can add additional execution info besides the pinotQuery object -- 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]
