fx19880617 edited a comment on pull request #9954: URL: https://github.com/apache/incubator-superset/pull/9954#issuecomment-637743106
> > No need to remove groupby fields from selection list anymore. > > Is this because it's been addressed in a newer version of Pinot? Do we not case about people on older versions? [Genuine questions] idk how the Pinot community distribution across versions is... Thanks for bring this up! In general, it won't change the behavior for old users as old pinot just ignores those fields, no matter the presence and always assume they are there. That's the reason we removed them to make query looks more like Pinot Style. This code change is for upgrade to use new Pinot Client `pinotdb-pypi` >=0.3.1, which leverages new Pinot SQL Endpoint(available on Pinot 0.3.0+) Also if it's possible to know the pinotdb version, then we could potentially keep the old behavior for old users and only enable this for users with newer pinotdb version. ---------------------------------------------------------------- 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]
