siddharthteotia commented on a change in pull request #4535: Implement DISTINCT clause URL: https://github.com/apache/incubator-pinot/pull/4535#discussion_r315383718
########## File path: pinot-core/src/main/java/org/apache/pinot/core/query/aggregation/AggregationFunctionContext.java ########## @@ -54,4 +59,16 @@ public String getColumn() { public String getAggregationColumnName() { return _aggregationFunction.getColumnName(_column); } + + public void setProjectedColumnTypes(final List<FieldSpec.DataType> projectedColumnTypes) { Review comment: The source of information is actually coming from metadata as mentioned here https://github.com/apache/incubator-pinot/pull/4535#discussion_r315382449 Just stashing it away in function context to be able to use it during construction of DataTable in InstanceResponeBlock ---------------------------------------------------------------- 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 With regards, Apache Git Services --------------------------------------------------------------------- To unsubscribe, e-mail: commits-unsubscr...@pinot.apache.org For additional commands, e-mail: commits-h...@pinot.apache.org