Skip to content

Commit 2ff1516

Browse files
Fix SearchWithRandomIOExceptionsIT tripping assertion in RefreshFieldHasValueListener
We're in some cases tripping an assertion (`assertSearcherIsWarmedUp`) when we run the logic and no refresh actually happened because of induced exceptions. This really should only run if the refresh actually went through in any case. fixes elastic#106752
1 parent b3b4214 commit 2ff1516

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

server/src/main/java/org/elasticsearch/index/shard/IndexShard.java

+1-1
Original file line numberDiff line numberDiff line change
@@ -4003,7 +4003,7 @@ public void beforeRefresh() {}
40034003

40044004
@Override
40054005
public void afterRefresh(boolean didRefresh) {
4006-
if (enableFieldHasValue) {
4006+
if (enableFieldHasValue && didRefresh) {
40074007
try (Engine.Searcher hasValueSearcher = getEngine().acquireSearcher("field_has_value")) {
40084008
setFieldInfos(FieldInfos.getMergedFieldInfos(hasValueSearcher.getIndexReader()));
40094009
} catch (AlreadyClosedException ignore) {

0 commit comments

Comments
 (0)