From 6233e32ab3e351834951de74ce2ddcb668bc8271 Mon Sep 17 00:00:00 2001 From: Tanguy Leroux Date: Thu, 7 May 2020 11:11:44 +0200 Subject: [PATCH] Fix SearchableSnapshotDirectoryTests.testIndexSearcher() (#56275) Closes #56233 --- .../index/store/BaseSearchableSnapshotIndexInput.java | 3 ++- .../index/store/SearchableSnapshotDirectoryTests.java | 1 - 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/x-pack/plugin/searchable-snapshots/src/main/java/org/elasticsearch/index/store/BaseSearchableSnapshotIndexInput.java b/x-pack/plugin/searchable-snapshots/src/main/java/org/elasticsearch/index/store/BaseSearchableSnapshotIndexInput.java index 01d9e92eafe..dccc29e3596 100644 --- a/x-pack/plugin/searchable-snapshots/src/main/java/org/elasticsearch/index/store/BaseSearchableSnapshotIndexInput.java +++ b/x-pack/plugin/searchable-snapshots/src/main/java/org/elasticsearch/index/store/BaseSearchableSnapshotIndexInput.java @@ -145,7 +145,8 @@ public abstract class BaseSearchableSnapshotIndexInput extends BufferedIndexInpu // Unit tests access the blob store on the main test thread; simplest just to permit this rather than have them override this // method somehow. - || threadName.startsWith("TEST-") : "current thread [" + Thread.currentThread() + "] may not read " + fileInfo; + || threadName.startsWith("TEST-") + || threadName.startsWith("LuceneTestCase") : "current thread [" + Thread.currentThread() + "] may not read " + fileInfo; return true; } diff --git a/x-pack/plugin/searchable-snapshots/src/test/java/org/elasticsearch/index/store/SearchableSnapshotDirectoryTests.java b/x-pack/plugin/searchable-snapshots/src/test/java/org/elasticsearch/index/store/SearchableSnapshotDirectoryTests.java index f3e2ee5b023..481bc66e230 100644 --- a/x-pack/plugin/searchable-snapshots/src/test/java/org/elasticsearch/index/store/SearchableSnapshotDirectoryTests.java +++ b/x-pack/plugin/searchable-snapshots/src/test/java/org/elasticsearch/index/store/SearchableSnapshotDirectoryTests.java @@ -138,7 +138,6 @@ public class SearchableSnapshotDirectoryTests extends ESTestCase { ); } - @AwaitsFix(bugUrl = "https://github.com/elastic/elasticsearch/issues/56233") public void testIndexSearcher() throws Exception { testDirectories((directory, snapshotDirectory) -> { try (DirectoryReader reader = DirectoryReader.open(directory)) {