You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@pinot.apache.org by GitBox <gi...@apache.org> on 2020/12/01 04:33:40 UTC

[GitHub] [incubator-pinot] Aka-shi opened a new issue #6303: [pinot] Throw error when column doesn't exist during aggregations

Aka-shi opened a new issue #6303:
URL: https://github.com/apache/incubator-pinot/issues/6303


   As of now, when we call an aggregate function on a column which doesn't exist in the pinot table, the query returns the default value for that aggregation type instead of throwing `column doesn't exist` or `column cannot be found in <table>` kind of error. This would be misleading for the user as the query will succeed. 
   
   Example Query:
   `select max(blah) from transcripts` - this return `-infinity`.


----------------------------------------------------------------
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



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


[GitHub] [incubator-pinot] kishoreg commented on issue #6303: [pinot] Throw error when column doesn't exist during aggregations

Posted by GitBox <gi...@apache.org>.
kishoreg commented on issue #6303:
URL: https://github.com/apache/incubator-pinot/issues/6303#issuecomment-736216130


   @jackjlli there was some implementation on this right?


----------------------------------------------------------------
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



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


[GitHub] [incubator-pinot] jackjlli commented on issue #6303: [pinot] Throw error when column doesn't exist during aggregations

Posted by GitBox <gi...@apache.org>.
jackjlli commented on issue #6303:
URL: https://github.com/apache/incubator-pinot/issues/6303#issuecomment-736788717


   Correct, we have a PR for that: https://github.com/apache/incubator-pinot/pull/6066


----------------------------------------------------------------
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



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