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

Change subject: WIP
......................................................................


Patch Set 3:

(4 comments)

http://gerrit.cloudera.org:8080/#/c/9980/2/docs/topics/impala_hints.xml
File docs/topics/impala_hints.xml:

http://gerrit.cloudera.org:8080/#/c/9980/2/docs/topics/impala_hints.xml@59
PS2, Line 59:         Creating a table based on column definitions from another 
table and
            :         copying data from the source table,
            :       </li>
            :     </ul>
            :
            :     <p conref="../shared/impala_common.xml#
> These two use cases are practically the same, as CTAS is practically a CREA
Done


http://gerrit.cloudera.org:8080/#/c/9980/2/docs/topics/impala_hints.xml@287
PS2, Line 287: codeph>/* +NOSHUFFLE */</codeph>
             :           hint would have any effect.</li>
> I think that it would be clearer if this was explained in the SHUFFLE parag
Done


http://gerrit.cloudera.org:8080/#/c/9980/2/docs/topics/impala_hints.xml@319
PS2, Line 319: ut how
> I think that "by each partitioning column" would be clearer.
Done


http://gerrit.cloudera.org:8080/#/c/9980/2/docs/topics/impala_hints.xml@328
PS2, Line 328:       during the query.
             :     </p>
             :
             :     <p>
             :       <b>Suggestions versus dir
> Since IMPALA-5293 (Impala 3.x only) the default is CLUSTERED behavior for H
Done



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

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I01057d59127d6193bdb69a2a181a386a039e08f5
Gerrit-Change-Number: 9980
Gerrit-PatchSet: 3
Gerrit-Owner: Alex Rodoni <[email protected]>
Gerrit-Reviewer: Alex Behm <[email protected]>
Gerrit-Reviewer: Alex Rodoni <[email protected]>
Gerrit-Reviewer: Csaba Ringhofer <[email protected]>
Gerrit-Reviewer: Impala Public Jenkins <[email protected]>
Gerrit-Reviewer: Lars Volker <[email protected]>
Gerrit-Reviewer: Thomas Tauber-Marshall <[email protected]>
Gerrit-Comment-Date: Wed, 11 Apr 2018 17:23:19 +0000
Gerrit-HasComments: Yes

Reply via email to