Alex Behm has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/9211 )

Change subject: IMPALA-5191, IMPALA-6415: [DOCS] Document breaking change of 
alias and ordinal substitution
......................................................................


Patch Set 3:

(4 comments)

http://gerrit.cloudera.org:8080/#/c/9211/3/docs/shared/impala_common.xml
File docs/shared/impala_common.xml:

http://gerrit.cloudera.org:8080/#/c/9211/3/docs/shared/impala_common.xml@664
PS3, Line 664:       </codeph>, and <codeph>ORDER BY</codeph> clauses. This 
means that from version 3.0 Impala doesn't allow
I think we should mention that we made this change to be more 
standard-conforming and consistent with other SQL DBMS.

There's also some duplication in words that can be improved, here's a slightly 
modified version to consider:

>From Impala 3.0, the alias substitution logic in the GROUP BY, HAVING, and 
>ORDER BY CLAUSES has become more consistent with standard SQL behavior, as 
>follows. Aliases are now only legal at the top level, and not in 
>subexpressions. The following statements are allowed:


http://gerrit.cloudera.org:8080/#/c/9211/3/docs/shared/impala_common.xml@698
PS3, Line 698:       You can also refer to the columns by their ordinals in the 
<codeph>SELECT</codeph> clause. Impala
You can also refer to SELECT-list items by their ordinal position.


http://gerrit.cloudera.org:8080/#/c/9211/3/docs/shared/impala_common.xml@700
PS3, Line 700:       </codeph> clauses. However, from Impala 3.0, ordinals can 
only be used at the top level. For
remove howeber


http://gerrit.cloudera.org:8080/#/c/9211/3/docs/shared/impala_common.xml@701
PS3, Line 701:       example, the followings are allowed:
the following statements are allowed



--
To view, visit http://gerrit.cloudera.org:8080/9211
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I558230d07212da62d2cd12e07a52ceba03e980a8
Gerrit-Change-Number: 9211
Gerrit-PatchSet: 3
Gerrit-Owner: Zoltan Borok-Nagy <[email protected]>
Gerrit-Reviewer: Alex Behm <[email protected]>
Gerrit-Reviewer: Alex Rodoni <[email protected]>
Gerrit-Reviewer: John Russell <[email protected]>
Gerrit-Reviewer: Zoltan Borok-Nagy <[email protected]>
Gerrit-Comment-Date: Wed, 14 Mar 2018 21:56:21 +0000
Gerrit-HasComments: Yes

Reply via email to