Github user dereksabryfb commented on the pull request:
https://github.com/apache/spark/pull/10052#issuecomment-162719598
Thanks for the feedback! I'm making the changes you suggested. With respect
to the 'ORDER BY' clause, this looks to be semi-handled by the
ResolveSortReferences in the Analyzer; because in standard SQL literals are
allowed in the order by clause, this rule just sorts by the literal value '1';
In HiveQL, '1' is interpreted as a column in the same way it is in the group by
clause.
I can add the case for a Sort() with an IntegerType. I assume there is no
one currently relying on how 'sort by 1' currently functions.
---
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]
