fx19880617 commented on pull request #5456: URL: https://github.com/apache/incubator-pinot/pull/5456#issuecomment-635100955
> I think the real issue is BrokerRequest is incapable of storing expressions correctly, it converts them to String. This seems like a temp workaround (approving it on that pretext). The ideal fix is to move to PinotQuery. Correct, this is also mention by @kishoreg , I will soon started move query processing from brokerRequest to PinotQuery. ---------------------------------------------------------------- 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. 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
