You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@pulsar.apache.org by GitBox <gi...@apache.org> on 2021/10/30 16:14:25 UTC

[GitHub] [pulsar] lhotari commented on a change in pull request #12547: [Tests] Reduce Pulsar IO integration test RAM requirements to prevent system OOM (exit code 137) in GitHub Actions CI

lhotari commented on a change in pull request #12547:
URL: https://github.com/apache/pulsar/pull/12547#discussion_r739670217



##########
File path: tests/integration/src/test/java/org/apache/pulsar/tests/integration/containers/ElasticSearchContainer.java
##########
@@ -23,24 +23,24 @@
 import java.util.Optional;
 
 public class ElasticSearchContainer extends ChaosContainer<ElasticSearchContainer> {
-    
+
     public static final String NAME = "ElasticSearch";
     static final Integer[] PORTS = { 9200, 9300 };
 
     public static final String IMAGE_NAME = Optional.ofNullable(System.getenv("ELASTICSEARCH_IMAGE"))
             .orElse("docker.elastic.co/elasticsearch/elasticsearch:7.15.0");
 
     public ElasticSearchContainer(String clusterName) {
-        super(clusterName, IMAGE_NAME);       
+        super(clusterName, IMAGE_NAME);
     }
-    
+
     @Override
     protected void configure() {
         super.configure();
         this.withNetworkAliases(NAME)
             .withExposedPorts(PORTS)
             .withEnv("discovery.type", "single-node")
-            .withEnv("ES_JAVA_OPTS", "-Xms2g -Xmx2g")
+            .withEnv("ES_JAVA_OPTS", "-Xms512m -Xmx1500m")

Review comment:
       I guess it would be worth specifying some value for min heap size? I'll merge since this is flaky. We can revisit later if the problem persists.




-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@pulsar.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org