Merge pull request #1310 from iceljc/master

remove try catch

Author: iceljc

src/Plugins/BotSharp.Plugin.KnowledgeBase/Graph/GraphKnowledgeService.cs

@@ -20,17 +20,9 @@ public GraphKnowledgeService(
 
     public async Task<GraphQueryResult> ExecuteQueryAsync(string query, GraphQueryOptions? options = null)
     {
-        try
-        {
-            var db = GetGraphDb(options?.Provider);
-            var result = await db.ExecuteQueryAsync(query, options);
-            return result;
-        }
-        catch (Exception ex)
-        {
-            _logger.LogError(ex, $"Error when searching graph knowledge (Query: {query}).");
-            return new GraphQueryResult();
-        }
+        var db = GetGraphDb(options?.Provider);
+        var result = await db.ExecuteQueryAsync(query, options);
+        return result;
     }
 
 

src/Plugins/BotSharp.Plugin.Membase/GraphDb/MembaseGraphDb.cs

@@ -55,7 +55,7 @@ public async Task<GraphQueryResult> ExecuteQueryAsync(string query, GraphQueryEx
         {
             var argLogs = args.Select(x => (new KeyValue(x.Key, x.Value.ConvertToString(BotSharpOptions.defaultJsonOptions))).ToString());
             _logger.LogError(ex, $"Error when executing query in {Provider} graph db. (Query: {query}), (Argments: \r\n{string.Join("\r\n", argLogs)})");
-            return new();
+            throw;
         }
     }
 }