villebro commented on pull request #13434: URL: https://github.com/apache/superset/pull/13434#issuecomment-791617426
> Thanks for the headsup, @villebro ! > > Another thing I noticed is that it's not always correct to assume "columns" with no "metrics" requires no aggregation. E.g. when table chart selects `groupby` with no metrics in aggregation mode, the output should be unique combination of the groups instead of raw records. This is mentioned as the second problem in #13228 . > > Therefore we can not really deprecate `groupby` in `QueryObject`. Or we will need to add at least an `is_aggregate` flag is `groupby` is to be cleaned up. Yes, I've been planning on that - I was thinking along the lines of an `aggregation_type`, which could be `none`, `distinct` and `groupby`, where any aggregate expression would force `groupby`, but in the absence of aggregates, we would default to `none`, but would provide the option to do `distinct`. ---------------------------------------------------------------- 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: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
