Lars Volker has posted comments on this change.

Change subject: IMPALA-3909: [DOCS] Add general info about Parquet min/max 
optimization
......................................................................


Patch Set 2: Code-Review+1

(2 comments)

LGTM, only nits.

http://gerrit.cloudera.org:8080/#/c/7068/2/docs/topics/impala_parquet.xml
File docs/topics/impala_parquet.xml:

PS2, Line 367: include
nit: includes


Line 369:         a particular Parquet file has a minimum value of 1 and a 
maximum value of 100, then 
nit: trailing whitespace


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

Gerrit-MessageType: comment
Gerrit-Change-Id: I5fd5f7b157024f6089af7feffcb538c160bb130d
Gerrit-PatchSet: 2
Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-Owner: John Russell <jruss...@cloudera.com>
Gerrit-Reviewer: John Russell <jruss...@cloudera.com>
Gerrit-Reviewer: Lars Volker <l...@cloudera.com>
Gerrit-Reviewer: Mostafa Mokhtar <mmokh...@cloudera.com>
Gerrit-Reviewer: Tim Armstrong <tarmstr...@cloudera.com>
Gerrit-HasComments: Yes

Reply via email to