[spambayes-dev] [ spambayes-Patches-756413 ] Add a SQL-based storage class

[SourceForge.net]

Patches item #756413, was opened at 2003-06-17 23:23
Message generated for change (Comment added) made by montanaro
You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=498105&aid=756413&group_id=61702

Category: None
Group: None
Status: Open
Resolution: None
Priority: 1
Submitted By: Tony Meyer (anadelonbrin)
Assigned to: Tony Meyer (anadelonbrin)
Summary: Add a SQL-based storage class

Initial Comment:
The attached patches add a SQL based storage class 
that can be used via pop3proxy.  It assumes that the 
user has mySQL and MySQLdb installed, and has already 
set up the table as suggested on the spambayes mailing 
list.

There is a great deal of room for improvement here.  In 
particular, having a separate table for the tokens is 
probably a waste (as discussed somewhat abstractly on 
the list).

If anyone wants an SQL solution, here it is.  This should 
be able to be patched into the current cvs for a 
reasonable while yet.

If you do end up using this, please let the list know.  If a 
suitably large number of people do (at least 2!), then it 
could be checked in to the main code.

----------------------------------------------------------------------

>Comment By: Skip Montanaro (montanaro)
Date: 2003-06-18 06:54

Message:
Logged In: YES 
user_id=44345

A couple quick observations without having tried the code:
* the diffs all seem to be reversed and are not context
  diffs, so it will be tough to apply them if the surrounding
  code changes at all

* don't you need to specify a database name in the connect
  call?

* maybe it should be called MySQLClassifier to admit other
  SQL implementations later.  For the short term, perhaps
  a dummy SQLClassifier could inherit from classifier.Classifier
  and be the base class for MySQLClassifier.


----------------------------------------------------------------------

You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=498105&aid=756413&group_id=61702