John Russell has posted comments on this change.

Change subject: [DOCS] Add doc for MT_DOP query option.
......................................................................


Patch Set 4:

(6 comments)

Addressed comments.

http://gerrit.cloudera.org:8080/#/c/5652/1/docs/topics/impala_mt_dop.xml
File docs/topics/impala_mt_dop.xml:

Line 38:       <indexterm audience="hidden">MT_DOP query option</indexterm>
> We should mention somewhere that COMPUTE [INCREMENTAL] STATS on Parquet tab
Done


Line 39:       Sets the degree of intra-node parallelism used for certain 
operations that
> Should we mention the operations where mt_dop applies?
Done


Line 42:       memory usage, and CPU usage during statement processing.
> I would reword to "ideal balance between response time, memory and CPU". 
Done


Line 43:     </p>
> I think it would make sense to document (for now) that many queries don't s
Done


http://gerrit.cloudera.org:8080/#/c/5652/3/docs/topics/impala_mt_dop.xml
File docs/topics/impala_mt_dop.xml:

Line 39:       Sets the degree of intra-node parallelism used for certain 
operations that
> of intra-node parallelism
Done


Line 67:             Queries with execution plans containing only scan and 
aggregation operators,
> or local joins that do not need data exchanges (e.g. for nested types)
Done


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

Gerrit-MessageType: comment
Gerrit-Change-Id: Ife2786532b425af6d230074f1c0b5c7dcb2b8a92
Gerrit-PatchSet: 4
Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-Owner: John Russell <jruss...@cloudera.com>
Gerrit-Reviewer: Alex Behm <alex.b...@cloudera.com>
Gerrit-Reviewer: John Russell <jruss...@cloudera.com>
Gerrit-Reviewer: Mostafa Mokhtar <mmokh...@cloudera.com>
Gerrit-Reviewer: Tim Armstrong <tarmstr...@cloudera.com>
Gerrit-HasComments: Yes

Reply via email to