You are viewing a plain text version of this content. The canonical link for it is here.

Posted to dev@superset.apache.org by GitBox <gi...@apache.org> on 2017/11/28 05:23:50 UTC

[GitHub] mistercrunch commented on issue #3944: Removing Count(*) from a Druid datasource selects a random metric by default

mistercrunch commented on issue #3944: Removing Count(*) from a Druid datasource selects a random metric by default
URL: https://github.com/apache/incubator-superset/issues/3944#issuecomment-347417301
 
 
   The `COUNT(*)` is not completely irrelevant and is a decent fallback default metric. I don't think we can assume that there will always be a `count` field to sum up either though.
   
   I agree that users should be able to define a default metric other than `COUNT(*)`. That applies to both SQLAlchemy and Druid.

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on 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