Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix IndexShardTests.testRestoreShardFromRemoteStore on Windows. #5399

Merged
merged 1 commit into from
Nov 30, 2022
Merged
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Original file line number Diff line number Diff line change
Expand Up @@ -2764,8 +2764,21 @@ public void testRestoreShardFromRemoteStore() throws IOException {

// Delete files in store directory to restore from remote directory
Directory storeDirectory = target.store().directory();

for (String file : storeDirectory.listAll()) {
storeDirectory.deleteFile(file);
// Windows has buggy File delete logic where AccessDeniedExceptions
// are thrown when there is an open file handle on a particular file. FSDirectory attempts to resolve this with hacks by
// swallowing the exceptions and moving the file to a pending delete state
// to retry in the future while being filtered from listAll invocations.
// However, this logic is also buggy and after the first delete attempt we are left in a state where the file is still on disk
// and not pending delete.
// A second attempt to delete the file will properly move it to pending deletion, and be filtered from listAll.
if (Arrays.asList(storeDirectory.listAll()).contains(file) && storeDirectory.getPendingDeletions().contains(file) == false) {
logger.info("File {} was not deleted and is not pending delete, attempting delete again...", file);
storeDirectory.deleteFile(file);
assertTrue(storeDirectory.getPendingDeletions().contains(file));
}
}

assertEquals(0, storeDirectory.listAll().length);
Expand Down