Skip to content

Commit eecb45c

Browse files
authored
[Maintenance]Enable indices.breaker.total.use_real_memory setting for integTest cluster (#2395)
Enable indices.breaker.total.use_real_memory setting for integTest Cluster vai build.gradle for catching heap CB in local integration testing. Signed-off-by: Rajvaibhav Rahane <rrahane@amazon.com>
1 parent 7e817f2 commit eecb45c

File tree

2 files changed

+12
-0
lines changed

2 files changed

+12
-0
lines changed

CHANGELOG.md

+1
Original file line numberDiff line numberDiff line change
@@ -47,4 +47,5 @@ The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/),
4747
* Added Lucene BWC tests (#2313)[https://github.com/opensearch-project/k-NN/pull/2313]
4848
* Upgrade jsonpath from 2.8.0 to 2.9.0[2325](https://github.com/opensearch-project/k-NN/pull/2325)
4949
* Bump Faiss commit from 1f42e81 to 0cbc2a8 to accelerate hamming distance calculation using _mm512_popcnt_epi64 intrinsic and also add avx512-fp16 instructions to boost performance [#2381](https://github.com/opensearch-project/k-NN/pull/2381)
50+
* Enabled indices.breaker.total.use_real_memory setting via build.gradle for integTest Cluster to catch heap CB in local ITs and github CI actions [#2395](https://github.com/opensearch-project/k-NN/pull/2395/)
5051
### Refactoring

build.gradle

+11
Original file line numberDiff line numberDiff line change
@@ -167,6 +167,15 @@ ext {
167167
)
168168
cluster.setSecure(true)
169169
}
170+
propertyKeys = [
171+
breaker: [
172+
useRealMemory: 'tests.opensearch.indices.breaker.total.use_real_memory'
173+
]
174+
]
175+
}
176+
177+
def getBreakerSetting() {
178+
return System.getProperty(propertyKeys.breaker.useRealMemory, 'true')
170179
}
171180

172181
allprojects {
@@ -399,6 +408,7 @@ integTest {
399408
if (System.getProperty("test.debug") != null) {
400409
jvmArgs '-agentlib:jdwp=transport=dt_socket,server=n,suspend=y,address=8000'
401410
}
411+
systemProperty propertyKeys.breaker.useRealMemory, getBreakerSetting()
402412
}
403413

404414
testClusters.integTest {
@@ -428,6 +438,7 @@ testClusters.integTest {
428438
}
429439
}
430440
systemProperty("java.library.path", "$rootDir/jni/release")
441+
systemProperty propertyKeys.breaker.useRealMemory, getBreakerSetting()
431442
}
432443

433444
task integTestRemote(type: RestIntegTestTask) {

0 commit comments

Comments
 (0)