siddharthteotia edited a comment on pull request #5483: URL: https://github.com/apache/incubator-pinot/pull/5483#issuecomment-642662988
It seems like we are adding some duplicate code (like Predicate, FilterInfo). We already have them. I agree that on server we can move to QueryRequest and build all the execution related information once (segment independent). But that can also be done while we continue to leverage existing interfaces and POJOs. We already have Expression, FilterQueryTree, Predicate etc. Why do we need a new implementatio? ---------------------------------------------------------------- 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
