John Russell has posted comments on this change.

Change subject: IMPALA-5359: [DOCS] Document SORT BY syntax for CREATE TABLE 
and ALTER TABLE
......................................................................


Patch Set 1:

(5 comments)

Addressed all the comments from Lars.

http://gerrit.cloudera.org:8080/#/c/6981/1/docs/topics/impala_create_table.xml
File docs/topics/impala_create_table.xml:

Line 388:       <codeph>CREATE TABLE AS SELECT</codeph> operation. Creating 
data files that are
> I think it's important to understand that the source table property does no
OK. I will mention that in a subsequent paragraph talking about cases where 
SORT BY does or doesn't apply.


Line 389:       sorted is most useful for Parquet tables, where the metadata 
includes the minimum and
> Here it should be clear that the information is stored in the file metadata
Done


Line 390:       maximum values for each column in each data file. Grouping data 
values together
> Technically, statistics are stored per RowGroup. Impala only writes 1 rowgr
Done


Line 400:       evident with Parquet tables.
> We could mention here, that other file formats don't have statistics inside
Done


Line 412:       tools that creat HDFS files, Impala does not guarantee or rely 
on the data being
> typo
Done


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

Gerrit-MessageType: comment
Gerrit-Change-Id: Icd571cd8840368edb327d16d27192458838ef234
Gerrit-PatchSet: 1
Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-Owner: John Russell <[email protected]>
Gerrit-Reviewer: Alan Choi <[email protected]>
Gerrit-Reviewer: Alex Behm <[email protected]>
Gerrit-Reviewer: John Russell <[email protected]>
Gerrit-Reviewer: Lars Volker <[email protected]>
Gerrit-HasComments: Yes

Reply via email to