Avoid using a waiting db for single-message-only transports. Performance
authorJeremy Harris <jgh146exb@wizmail.org>
Tue, 11 Sep 2012 22:11:16 +0000 (23:11 +0100)
committerJeremy Harris <jgh146exb@wizmail.org>
Tue, 11 Sep 2012 22:11:16 +0000 (23:11 +0100)
bug 1262 and patch from Paul Fisher.  Testcase 0288 exercises.

doc/doc-txt/ChangeLog
src/src/transports/smtp.c

index c528ada..f9eaaf7 100644 (file)
@@ -68,6 +68,9 @@ JH/05 Permit multiple router/transport headers_add/remove lines.
 
 JH/06 Add dnsdb pseudo-lookup "a+" to do an "aaaa" + "a" combination.
 
+JH/07 Avoid using a waiting database for a single-message-only transport.
+      Performance patch from Paul Fisher.
+
 Exim version 4.80
 -----------------
 
index f7af921..36a053f 100644 (file)
@@ -3114,9 +3114,12 @@ for (addr = addrlist; addr != NULL; addr = addr->next)
 /* Update the database which keeps information about which messages are waiting
 for which hosts to become available. For some message-specific errors, the
 update_waiting flag is turned off because we don't want follow-on deliveries in
-those cases. */
+those cases.  If this transport instance is explicitly limited to one message
+per connection, follow-on deliveries are not possible, and there's no need
+to create/update a waiting database. */
 
-if (update_waiting) transport_update_waiting(hostlist, tblock->name);
+if (update_waiting && tblock->connection_max_messages != 1)
+  transport_update_waiting(hostlist, tblock->name);
 
 END_TRANSPORT: