From: Peter van Dijk <peter.van.dijk@netherlabs.nl>
Date: Fri, 19 Oct 2012 11:30:31 +0000 (+0000)
Subject: revert the actual lock bit of r2807, as it causes more parallel transfers of the... 
X-Git-Tag: auth-3.2-rc1~64
X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=f4ce804f7bfd7ff346c4851261f6b791d5aa34c2;p=pdns

revert the actual lock bit of r2807, as it causes more parallel transfers of the same domain than it prevented

git-svn-id: svn://svn.powerdns.com/pdns/trunk/pdns@2818 d19b8d6e-7fed-0310-83ef-9ca221ded41b
---

diff --git a/pdns/communicator.cc b/pdns/communicator.cc
index 28aa63c2b..ddc9fbbef 100644
--- a/pdns/communicator.cc
+++ b/pdns/communicator.cc
@@ -45,6 +45,7 @@ void CommunicatorClass::retrievalLoopThread(void)
         continue;
 	
       sr=d_suckdomains.front();
+      d_suckdomains.pop_front();
     }
     try {
       suck(sr.domain,sr.master);
@@ -52,12 +53,6 @@ void CommunicatorClass::retrievalLoopThread(void)
     catch(AhuException& ae) {
       cerr<<"Error: "<<ae.reason<<endl;
     }
-
-    {
-      Lock l(&d_lock);
-      domains_by_name_t& uqIndex = d_suckdomains.get<IDTag>();
-      uqIndex.erase(sr);
-    }
   }
 
 }
diff --git a/pdns/slavecommunicator.cc b/pdns/slavecommunicator.cc
index 04bf34d56..869d8102f 100644
--- a/pdns/slavecommunicator.cc
+++ b/pdns/slavecommunicator.cc
@@ -518,7 +518,6 @@ void CommunicatorClass::slaveRefresh(PacketHandler *P)
       // remove unfresh domains already queued for AXFR, no sense polling them again
       sr.master=*di.masters.begin();
       if(nameindex.count(sr)) {
-        L<<Logger::Warning<<"Domain "<<sr.domain<<" already queued for AXFR."<<endl;
         continue;
       }
       DomainNotificationInfo dni;