From eb630372ba3d58d4de4e1b5c8da6ab0cfab092a4 Mon Sep 17 00:00:00 2001 From: maliming <6908465+maliming@users.noreply.github.com> Date: Mon, 25 May 2020 11:04:01 +0800 Subject: [PATCH] Write error logs instead of throwing exceptions when Elasitc Search has errors Related #4042 --- .../FullSearch/Elastic/ElasticDocumentFullSearch.cs | 10 ++++++++-- 1 file changed, 8 insertions(+), 2 deletions(-) diff --git a/modules/docs/src/Volo.Docs.Domain/Volo/Docs/Documents/FullSearch/Elastic/ElasticDocumentFullSearch.cs b/modules/docs/src/Volo.Docs.Domain/Volo/Docs/Documents/FullSearch/Elastic/ElasticDocumentFullSearch.cs index 854032ab2c..cbe5905d3a 100644 --- a/modules/docs/src/Volo.Docs.Domain/Volo/Docs/Documents/FullSearch/Elastic/ElasticDocumentFullSearch.cs +++ b/modules/docs/src/Volo.Docs.Domain/Volo/Docs/Documents/FullSearch/Elastic/ElasticDocumentFullSearch.cs @@ -3,6 +3,7 @@ using System.Collections.Generic; using System.Threading; using System.Threading.Tasks; using Elasticsearch.Net; +using Microsoft.Extensions.Logging; using Microsoft.Extensions.Options; using Nest; using Volo.Abp; @@ -14,10 +15,14 @@ namespace Volo.Docs.Documents.FullSearch.Elastic { private readonly IElasticClientProvider _clientProvider; private readonly DocsElasticSearchOptions _options; + private readonly ILogger _logger; - public ElasticDocumentFullSearch(IElasticClientProvider clientProvider, IOptions options) + public ElasticDocumentFullSearch(IElasticClientProvider clientProvider, + IOptions options, + ILogger logger) { _clientProvider = clientProvider; + _logger = logger; _options = options.Value; } @@ -215,7 +220,8 @@ namespace Volo.Docs.Documents.FullSearch.Elastic { if (!response.ApiCall.Success) { - throw response.ApiCall.OriginalException; + _logger.LogError(response.ApiCall.OriginalException, + "An error occurred in the elastic search api call."); } }