[impala] branch master updated: IMPALA-9177: Bump min-query-mem-limit for dockerised tests

[jo...@apache.org]

This is an automated email from the ASF dual-hosted git repository.

joemcdonnell pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/impala.git


The following commit(s) were added to refs/heads/master by this push:
     new 9c167c0  IMPALA-9177: Bump min-query-mem-limit for dockerised tests
9c167c0 is described below

commit 9c167c09640c1db104d6e68d900422d29c3076e2
Author: Joe McDonnell <jo...@cloudera.com>
AuthorDate: Wed Nov 20 16:06:16 2019 -0800

    IMPALA-9177: Bump min-query-mem-limit for dockerised tests
    
    A couple upstream dockerised test runs have seen
    TestTpchQuery.test_tpch query 18 on Kudu hit a memory limit.
    This is memory limit comes from the admission control
    settings, as admission control is enabled for the dockerised
    tests in IMPALA-8451.
    
    This bumps the min-query-mem-limit from 256MB to 300MB to
    avoid hitting this memory limit.
    
    Testing:
     - Ran GVO. Dockerised tests did not change in run length.
    
    Change-Id: I7f478b988a7bc2e7bd7422ceda52c6eddac04d0a
    Reviewed-on: http://gerrit.cloudera.org:8080/14759
    Reviewed-by: Tim Armstrong <ta...@cloudera.com>
    Tested-by: Impala Public Jenkins <im...@cloudera.com>
---
 fe/src/test/resources/minicluster-llama-site.xml | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/fe/src/test/resources/minicluster-llama-site.xml b/fe/src/test/resources/minicluster-llama-site.xml
index a835e7c..affc9a4 100644
--- a/fe/src/test/resources/minicluster-llama-site.xml
+++ b/fe/src/test/resources/minicluster-llama-site.xml
@@ -12,7 +12,7 @@
          so as not to reduce query concurrency but high enough that
          queries don't get starved for memory because of low estimates. -->
     <name>impala.admission-control.min-query-mem-limit.root.default</name>
-    <value>268435456</value><!--256MB-->
+    <value>314572800</value><!--300MB-->
   </property>
   <property>
     <!--Allow individual tests to set mem_limit to any value, if needed.-->