You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@spamassassin.apache.org by Mark Martinec <Ma...@ijs.si> on 2007/08/14 01:01:23 UTC

Docs question - file descriptors in parallel networks tests

(I'm just fixing some typos in comments and docs, this is what I came across):

The Conf.pm claims in a POD:


SpamAssassin's network rules are run in parallel.  This can cause overhead in
terms of the number of file descriptors required; it is recommended that the
minimum limit on file descriptors be raised to at least 256 for safety.


Is this still true? I don't think it is.

  Mark

Re: Docs question - file descriptors in parallel networks tests

Posted by Mark Martinec <Ma...@ijs.si>.
> I think it became no longer true as a result of the fix for bug 4260.
> It's easier to see reading the comments in its duplicate bug 4016
>
> http://issues.apache.org/SpamAssassin/show_bug.cgi?id=4260
> http://issues.apache.org/SpamAssassin/show_bug.cgi?id=4016

Thanks for the reference. Removed text in trunk:

$ svn -m 'Removed dated text about no. of file descr. in Conf.pm' ci
Sending        lib/Mail/SpamAssassin/Conf.pm
Committed revision 565649.

  Mark

Re: Docs question - file descriptors in parallel networks tests

Posted by Sidney Markowitz <si...@sidney.com>.
Mark Martinec wrote, On 14/8/07 11:01 AM:
> Is this still true? I don't think it is.

I think it became no longer true as a result of the fix for bug 4260.
It's easier to see reading the comments in its duplicate bug 4016

http://issues.apache.org/SpamAssassin/show_bug.cgi?id=4260
http://issues.apache.org/SpamAssassin/show_bug.cgi?id=4016


 -- sidney


Re: Docs question - file descriptors in parallel networks tests

Posted by Mark Martinec <Ma...@ijs.si>.
> I'm just fixing some typos in comments and docs

(a byproduct of my summer code audit)

If someone cares to check (in trunk):

$ svn -m 'In comments, POD and dbg strings:
          fix 70 typos, mistakes and similar' ci
Sending        lib/Mail/SpamAssassin/ArchiveIterator.pm
Sending        lib/Mail/SpamAssassin/Bayes.pm
Sending        lib/Mail/SpamAssassin/BayesStore/DBM.pm
Sending        lib/Mail/SpamAssassin/BayesStore/PgSQL.pm
Sending        lib/Mail/SpamAssassin/BayesStore/SQL.pm
Sending        lib/Mail/SpamAssassin/BayesStore.pm
Sending        lib/Mail/SpamAssassin/Client.pm
Sending        lib/Mail/SpamAssassin/Conf.pm
Sending        lib/Mail/SpamAssassin/Constants.pm
Sending        lib/Mail/SpamAssassin/DnsResolver.pm
Sending        lib/Mail/SpamAssassin/MailingList.pm
Sending        lib/Mail/SpamAssassin/Message/Node.pm
Sending        lib/Mail/SpamAssassin/Message.pm
Sending        lib/Mail/SpamAssassin/PerMsgStatus.pm
Sending        lib/Mail/SpamAssassin/Plugin/ASN.pm
Sending        lib/Mail/SpamAssassin/Plugin/DKIM.pm
Sending        lib/Mail/SpamAssassin/Plugin/DNSEval.pm
Sending        lib/Mail/SpamAssassin/Plugin/DomainKeys.pm
Sending        lib/Mail/SpamAssassin/Plugin/P595Body.pm
Sending        lib/Mail/SpamAssassin/Plugin/ReplaceTags.pm
Sending        lib/Mail/SpamAssassin/Plugin/Rule2XSBody.pm
Sending        lib/Mail/SpamAssassin/Plugin/SPF.pm
Sending        lib/Mail/SpamAssassin/Plugin/URIDNSBL.pm
Sending        lib/Mail/SpamAssassin/SQLBasedAddrList.pm
Sending        lib/Mail/SpamAssassin/Util/DependencyInfo.pm
Sending        lib/Mail/SpamAssassin/Util/Progress.pm
Sending        lib/Mail/SpamAssassin/Util/RegistrarBoundaries.pm
Sending        lib/Mail/SpamAssassin/Util/TieOneStringHash.pm
Sending        lib/Mail/SpamAssassin/Util.pm
Transmitting file data .............................
Committed revision 565567.

  Mark