Skip to content

Commit

Permalink
Lower heap memory allocation for test cluster, disable CB for all tes…
Browse files Browse the repository at this point in the history
…ts (#770)

Signed-off-by: Martin Gaievski <[email protected]>
  • Loading branch information
martin-gaievski authored Jun 4, 2024
1 parent 2b21110 commit 8705980
Show file tree
Hide file tree
Showing 6 changed files with 2 additions and 21 deletions.
1 change: 1 addition & 0 deletions CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -24,6 +24,7 @@ The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/),
### Bug Fixes
- Total hit count fix in Hybrid Query ([756](https://github.com/opensearch-project/neural-search/pull/756))
### Infrastructure
- Disable memory circuit breaker for integ tests ([#770](https://github.com/opensearch-project/neural-search/pull/770))
### Documentation
### Maintenance
### Refactoring
2 changes: 1 addition & 1 deletion build.gradle
Original file line number Diff line number Diff line change
Expand Up @@ -396,7 +396,7 @@ testClusters.integTest {

// Increase heap size from default of 512mb to 1gb. When heap size is 512mb, our integ tests sporadically fail due
// to ml-commons memory circuit breaker exception
jvmArgs("-Xms1g", "-Xmx4g")
jvmArgs("-Xms1g", "-Xmx2g")
}

// Remote Integration Tests
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -52,11 +52,6 @@ public void setUp() throws Exception {
updateClusterSettings();
}

@Override
public boolean isUpdateClusterSettings() {
return false;
}

/**
* Using search pipelines with config for l2 norm:
* {
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -93,11 +93,6 @@ public void setUp() throws Exception {
updateClusterSettings();
}

@Override
public boolean isUpdateClusterSettings() {
return false;
}

@Override
protected boolean preserveClusterUponCompletion() {
return true;
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -87,11 +87,6 @@ public void setUp() throws Exception {
updateClusterSettings();
}

@Override
public boolean isUpdateClusterSettings() {
return false;
}

@Override
protected boolean preserveClusterUponCompletion() {
return true;
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -89,11 +89,6 @@ public static void setUpCluster() {
instance.updateClusterSettings();
}

@Override
public boolean isUpdateClusterSettings() {
return false;
}

@Override
protected boolean preserveClusterUponCompletion() {
return true;
Expand Down

0 comments on commit 8705980

Please sign in to comment.