Minor doc nits re bug 1262.
authorPhil Pennock <pdp@exim.org>
Wed, 12 Sep 2012 00:14:42 +0000 (20:14 -0400)
committerPhil Pennock <pdp@exim.org>
Wed, 12 Sep 2012 00:14:42 +0000 (20:14 -0400)
Update src comment to be clearer about why it's safe for "state of this transport" to affect other deliveries.
Mention change in externally observable state in README.UPDATING.
Reference bugzilla entry in ChangeLog.
Update Paul's credit in ACKNOWLEDGMENTS.

doc/doc-txt/ChangeLog
src/ACKNOWLEDGMENTS
src/README.UPDATING
src/src/transports/smtp.c

index f9eaaf7..a4c10d1 100644 (file)
@@ -69,7 +69,8 @@ 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.
+      Performance patch from Paul Fisher.  Bugzilla 1262.
+
 
 Exim version 4.80
 -----------------
index a611a29..a66512f 100644 (file)
@@ -382,6 +382,7 @@ Maxim Dounin              Patch portability of accept() len
 Frank Elsner              Fixed build reliability by exporting LC_ALL=C
 Paul Fisher               Diagnosed smtp_cmd_buffer_size affecting GSSAPI SASL
                             initial response, raised buffer size
+                          Patch adjusting connection_max_messages wait-DB usage
 Oliver Fleischmann        Patches fixing compilation in older compilers
 Julian Gilbey             Helped improve userforward local_part_suffix docs
 Richard Godbee            Patch fixing usage fprintf
index b7406f4..5742a64 100644 (file)
@@ -32,6 +32,11 @@ Exim version 4.81
  * New option gnutls_enable_pkcs11 defaults false; if you have GnuTLS 2.12.0
    or later and do want PKCS11 modules to be autoloaded, then set this option.
 
+ * A per-transport wait-<name> database is no longer updated if the transport
+   sets "connection_max_messages" to 1, as it can not be used and causes
+   unnecessary serialisation and load.  External tools tracking the state of
+   Exim by the hints databases may need modification to take this into account.
+
 
 Exim version 4.80
 -----------------
index 36a053f..dc24e69 100644 (file)
@@ -3115,8 +3115,8 @@ for (addr = addrlist; addr != NULL; addr = addr->next)
 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.  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. */
+per connection then follow-on deliveries are not possible and there's no need
+to create/update the per-transport wait-<transport_name> database. */
 
 if (update_waiting && tblock->connection_max_messages != 1)
   transport_update_waiting(hostlist, tblock->name);