Github user dereksabryfb commented on the pull request:
https://github.com/apache/spark/pull/10052#issuecomment-163831807
I removed the offending test.
I found that there was no `AnalysisException` thrown even if there was an
explicit aggregate in the group by clause (e.g. select a from b group by
count(a)), and it would fail in the same way, so I added the check to
`CheckAnalysis` ; if you think this is out of the scope of this pull request
(since it isn't strictly to do with a number reference), I can create a new
task and attach just that commit to it.
Thanks again for your feedback.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at [email protected] or file a JIRA ticket
with INFRA.
---
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
