You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@trafficserver.apache.org by zw...@apache.org on 2017/11/07 14:34:09 UTC

[trafficserver] branch 7.1.x updated: Revert "Revert "Added stop and close to else in failover to close a single configured connection on failover.""

This is an automated email from the ASF dual-hosted git repository.

zwoop pushed a commit to branch 7.1.x
in repository https://gitbox.apache.org/repos/asf/trafficserver.git


The following commit(s) were added to refs/heads/7.1.x by this push:
     new 4d6d99e  Revert "Revert "Added stop and close to else in failover to close a single configured connection on failover.""
4d6d99e is described below

commit 4d6d99e4a5aef80c5fbd858448c8ff4a3348ca53
Author: Leif Hedstrom <zw...@apache.org>
AuthorDate: Tue Nov 7 07:33:00 2017 -0700

    Revert "Revert "Added stop and close to else in failover to close a single configured connection on failover.""
    
    This reverts commit a377284a4647282df222ff474706af9090724fbc.
    
    Turns out, master has devidated sufficiently, such that we do indeeed
    need to make this a separate PR / commit after all.
---
 iocore/dns/DNS.cc | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/iocore/dns/DNS.cc b/iocore/dns/DNS.cc
index 5c1522a..b77b190 100644
--- a/iocore/dns/DNS.cc
+++ b/iocore/dns/DNS.cc
@@ -645,6 +645,8 @@ DNSHandler::failover()
     switch_named(name_server);
   } else {
     ip_text_buffer buff;
+    con[name_server].eio.stop();
+    con[name_server].close();
     Warning("failover: connection to DNS server %s lost, retrying", ats_ip_ntop(&ip.sa, buff, sizeof(buff)));
   }
 }

-- 
To stop receiving notification emails like this one, please contact
['"commits@trafficserver.apache.org" <co...@trafficserver.apache.org>'].