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

Change subject: [DOCS] Clarify scope of STRAIGHT_JOIN hint
......................................................................


Patch Set 1:

(4 comments)

http://gerrit.cloudera.org:8080/#/c/8336/1//COMMIT_MSG
Commit Message:

http://gerrit.cloudera.org:8080/#/c/8336/1//COMMIT_MSG@7
PS1, Line 7: [DOCS] Clarify scope of STRAIGHT_JOIN hint
IMPALA-6083


http://gerrit.cloudera.org:8080/#/c/8336/1/docs/shared/impala_common.xml
File docs/shared/impala_common.xml:

http://gerrit.cloudera.org:8080/#/c/8336/1/docs/shared/impala_common.xml@2869
PS1, Line 2869:         in the topmost query block where the hint occurs. It 
does not affect the join order
"topmost" is confusing. It is simply the query block in which the hint appears. 
How about:

... affects the join order of table references in the query block containing 
the hint.


http://gerrit.cloudera.org:8080/#/c/8336/1/docs/shared/impala_common.xml@2870
PS1, Line 2870:         of any queries nested within those table references, 
such as subqueries or views.
of any nested queries, such as views, inline views or WHERE-clause subqueries.

(generally, not all nested queries are part of a table reference)


http://gerrit.cloudera.org:8080/#/c/8336/1/docs/shared/impala_common.xml@2872
PS1, Line 2872:         whichever top-level or nested queries need a specific 
join order.
apply the hint to all query blocks that need a fixed join order



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

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: Ibd8e4d8e5260f5a2b7b5c52dc3269bb373b9c76f
Gerrit-Change-Number: 8336
Gerrit-PatchSet: 1
Gerrit-Owner: John Russell <[email protected]>
Gerrit-Reviewer: Alex Behm <[email protected]>
Gerrit-Reviewer: Mala Chikka Kempanna <[email protected]>
Gerrit-Comment-Date: Thu, 19 Oct 2017 19:03:58 +0000
Gerrit-HasComments: Yes

Reply via email to