Skip to content

Commit e5438f8

Browse files
committed
Fix IndexStatsIT#testThrottleStats
`Engine.PauseLock#throttle` can be called when the lock is being throttled, so we can't guarantee that all permits are available before throttling. Resolve elastic#126359 See elastic#127173
1 parent d07ec0c commit e5438f8

File tree

2 files changed

+3
-4
lines changed

2 files changed

+3
-4
lines changed

muted-tests.yml

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -444,9 +444,9 @@ tests:
444444
- class: org.elasticsearch.xpack.esql.qa.mixed.EsqlClientYamlIT
445445
method: test {p0=esql/120_profile/avg 8.14 or after}
446446
issue: https://github.com/elastic/elasticsearch/issues/127879
447-
- class: org.elasticsearch.indices.stats.IndexStatsIT
448-
method: testThrottleStats
449-
issue: https://github.com/elastic/elasticsearch/issues/126359
447+
- class: org.elasticsearch.cluster.routing.allocation.decider.DiskThresholdDeciderIT
448+
method: testRestoreSnapshotAllocationDoesNotExceedWatermarkWithMultipleRestores
449+
issue: https://github.com/elastic/elasticsearch/issues/127787
450450
- class: org.elasticsearch.search.vectors.IVFKnnFloatVectorQueryTests
451451
method: testRandomWithFilter
452452
issue: https://github.com/elastic/elasticsearch/issues/127963

server/src/main/java/org/elasticsearch/index/engine/Engine.java

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -612,7 +612,6 @@ public Condition newCondition() {
612612
}
613613

614614
public void throttle() {
615-
assert semaphore.availablePermits() == Integer.MAX_VALUE;
616615
semaphore.acquireUninterruptibly(Integer.MAX_VALUE - allowThreads);
617616
}
618617

0 commit comments

Comments
 (0)