[GitHub] [incubator-pinot] npawar commented on issue #4870: Add getResultColumnName API to AggregationFunction to fetch sql compliant result name

[GitBox <gi...@apache.org>]

npawar commented on issue #4870: Add getResultColumnName API to AggregationFunction to fetch sql compliant result name
URL: https://github.com/apache/incubator-pinot/pull/4870#issuecomment-559280223
 
 
   > This is good to have for now since we don't support aliasing. But this should be typically solved at a very high level using a mapping table. Let's make sure that this will not make it difficult to implement aliasing later.
   
   I feel this is required. We have no other way to fetch the name like `avg(foo)`. Order by cannot understand `avg_foo`, as the TransformExpressionTree converts it to `avg(foo)`.
   I don't think this should affect ability to handle aliases later

----------------------------------------------------------------
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:
users@infra.apache.org


With regards,
Apache Git Services

---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@pinot.apache.org
For additional commands, e-mail: commits-help@pinot.apache.org