Tim Armstrong has posted comments on this change. ( http://gerrit.cloudera.org:8080/10346 )
Change subject: [DOCS] Removed the references to YARN as Impala does not support YARN ...................................................................... Patch Set 1: (2 comments) http://gerrit.cloudera.org:8080/#/c/10346/1/docs/topics/impala_mem_limit.xml File docs/topics/impala_mem_limit.xml: http://gerrit.cloudera.org:8080/#/c/10346/1/docs/topics/impala_mem_limit.xml@41 PS1, Line 41: When resource management is not enabled, defines the maximum amount of memory a query can allocate on each node. We can remove the "When resource management is not enabled, " bit, since it's referring to the YARN integration. http://gerrit.cloudera.org:8080/#/c/10346/1/docs/topics/impala_mem_limit.xml@64 PS1, Line 64: When resource management is enabled, the mechanism for this option I think we should just remove this whole paragraph since it's talking about the YARN integration. MEM_LIMIT is used for memory-based admission control but I think the behaviour there is probably better covered in the context of admission control. -- To view, visit http://gerrit.cloudera.org:8080/10346 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: Ifcea49b5859a2afbbbe99197e7818c30c7ba6d67 Gerrit-Change-Number: 10346 Gerrit-PatchSet: 1 Gerrit-Owner: Alex Rodoni <[email protected]> Gerrit-Reviewer: Impala Public Jenkins <[email protected]> Gerrit-Reviewer: Tim Armstrong <[email protected]> Gerrit-Comment-Date: Tue, 08 May 2018 20:19:54 +0000 Gerrit-HasComments: Yes
