You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@kudu.apache.org by gr...@apache.org on 2019/01/30 20:45:55 UTC

[kudu] 03/03: [gradle] Increase test JVMs heap size

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

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

commit 9ac3b295a9cf41316e100f5f1745d3c6120d6c48
Author: Hao Hao <ha...@cloudera.com>
AuthorDate: Thu Dec 27 16:41:00 2018 -0800

    [gradle] Increase test JVMs heap size
    
    The flume test RegexpKuduOperationsProducerParseErrorTest is flaky and
    throws OutOfMemoryError. As the default heap size for test task is 512m
    [1], this commit increases the maximum java heap size to 1g to avoid
    such error.
    
    [1]: https://docs.gradle.org/current/userguide/build_environment.html#sec:configuring_jvm_memory
    
    Change-Id: Iae86fe493a1fdc337d5ca8615b043b437a2a116d
    Reviewed-on: http://gerrit.cloudera.org:8080/12134
    Tested-by: Kudu Jenkins
    Reviewed-by: Adar Dembo <ad...@cloudera.com>
    Reviewed-by: Grant Henke <gr...@apache.org>
---
 java/gradle/tests.gradle | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/java/gradle/tests.gradle b/java/gradle/tests.gradle
index 9c51858..2f43d96 100644
--- a/java/gradle/tests.gradle
+++ b/java/gradle/tests.gradle
@@ -38,6 +38,8 @@ tasks.withType(Test) {
 
   // Enable assertions during tests.
   jvmArgs += "-enableassertions"
+  // Config 1g maximum java heap size for test JVMs.
+  jvmArgs += "-Xmx1g"
   // Hide the Java 9+ warnings about illegal reflective access
   if (JavaVersion.current().isJava9Compatible()) {
     jvmArgs += "--illegal-access=permit"