Skip to content

Commit 62fe090

Browse files
committed
Catch exception during indexing
1 parent 2003ec0 commit 62fe090

File tree

1 file changed

+15
-16
lines changed

1 file changed

+15
-16
lines changed

x-pack/plugin/searchable-snapshots/src/main/java/org/elasticsearch/blobstore/cache/BlobStoreCacheService.java

Lines changed: 15 additions & 16 deletions
Original file line numberDiff line numberDiff line change
@@ -316,28 +316,27 @@ public void onResponse(String s) {
316316
try (XContentBuilder builder = jsonBuilder()) {
317317
request.source(cachedBlob.toXContent(builder, ToXContent.EMPTY_PARAMS));
318318
}
319-
} catch (IOException e) {
319+
320+
client.index(request, new ActionListener<>() {
321+
@Override
322+
public void onResponse(IndexResponse indexResponse) {
323+
logger.trace("cache fill ({}): [{}]", indexResponse.status(), request.id());
324+
listener.onResponse(null);
325+
}
326+
327+
@Override
328+
public void onFailure(Exception e) {
329+
logger.debug(new ParameterizedMessage("failure in cache fill: [{}]", request.id()), e);
330+
listener.onFailure(e);
331+
}
332+
});
333+
} catch (Exception e) {
320334
logger.warn(
321335
new ParameterizedMessage("cache fill failure: [{}]", CachedBlob.generateId(repository, name, path, offset)),
322336
e
323337
);
324338
listener.onFailure(e);
325-
return;
326339
}
327-
328-
client.index(request, new ActionListener<>() {
329-
@Override
330-
public void onResponse(IndexResponse indexResponse) {
331-
logger.trace("cache fill ({}): [{}]", indexResponse.status(), request.id());
332-
listener.onResponse(null);
333-
}
334-
335-
@Override
336-
public void onFailure(Exception e) {
337-
logger.debug(new ParameterizedMessage("failure in cache fill: [{}]", request.id()), e);
338-
listener.onFailure(e);
339-
}
340-
});
341340
}
342341

343342
@Override

0 commit comments

Comments
 (0)