Jackie-Jiang commented on pull request #6027: URL: https://github.com/apache/incubator-pinot/pull/6027#issuecomment-695141063
Discussed with @siddharthteotia and @fx19880617 offline and we all agreed that `QueryExecutor` logic for streaming and non-streaming query will remain very similar, and the actual streaming handling is handled on the `Operator` level. Instead of changing the current `processQuery()` method, decide to add another method for the streaming query so that in the future we can easily split them if necessary. ---------------------------------------------------------------- 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
