Merge pull request #14 from yileicn/hotfix

优化Consul PollForChanges
yileicn · Aug 25, 2021 · b87b556 · b87b556
2 parents e5f76f7 + f31de6d
commit b87b556
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 2 deletions.
diff --git a/src/Grpc.Extension.Client/Options/JaegerOptions.cs b/src/Grpc.Extension.Client/Options/JaegerOptions.cs
@@ -12,7 +12,7 @@ public class JaegerOptions
         /// <summary>
         /// 是否启用Jaeger
         /// </summary>
-        public bool Enable { get; set; } = true;
+        public bool Enable { get; set; }
 
         /// <summary>
         /// 服务名

diff --git a/src/Grpc.Extension.Discovery/Consul/ConsulServiceDiscovery.cs b/src/Grpc.Extension.Discovery/Consul/ConsulServiceDiscovery.cs
@@ -31,7 +31,8 @@ public async Task<List<string>> GetEndpoints(string serviceName, string consulUr
             //更新LastIndex
             UpdateLastIndex(serviceName, res);
             //PollForChanges
-            _ = Task.Run(() => PollForChanges(serviceName, consulUrl, consulTag));
+            if (!_pollForChanges.Contains(serviceName))
+                _ = Task.Run(() => PollForChanges(serviceName, consulUrl, consulTag));
 
             return res.Response.Select(q => $"{q.Service.Address}:{q.Service.Port}").ToList();
         }