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

Reply via email to