Jackie-Jiang opened a new pull request #5605: URL: https://github.com/apache/incubator-pinot/pull/5605
## Description Replace BrokerRequest with QueryContext in Operators (server side) and DataTableReducers (broker side) The reason to put these 2 part in one PR is because they share the same code for selection queries The change is backward-compatible because it does not involve any change on the wiring layer Changes for QueryContext: - Select expressions will contain both aggregation and non-aggregation expressions so that the columns in the result table can be correctly ordered - Change method signature from `getAlias(ExpressionCOntext expression)` to `getAliasMap()` and return an unmodifiable map to make it easier to use ---------------------------------------------------------------- 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
