Darrell,
This looks great. Thanks a lot! I'm going to test this weekend to see
what's left to do. I also like the build approach you used.
In terms of configuring whether to store the entire message in the DB or the
headers in the DB and body in the file, why don't we create two classes
instead. The db:// would store the complete message in the DB and dbfile://
would store only the headers. Perhaps the two implementations could extend
from a single abstract class so the SQL statements only need to get defined
once (I'm not sure whether your resource parser handles inheritance).
Serge Knystautas
Loki Technologies
http://www.lokitech.com/
----- Original Message -----
From: "Darrell DeBoer" <[EMAIL PROTECTED]>
To: <[EMAIL PROTECTED]>
Sent: Friday, August 17, 2001 11:45 AM
Subject: [COMMIT] - More UserStore/MailStore stuff
> G'day all,
>
> I've just commited a bunch of code I've been working on over the last week
or
> so. It involves a bit of cleaning up the configuration for Users/Mail
> repositories and (I hope), better useability.
>
> This is all in the "userstore" proposal, but I can't see any reason why it
> can't be moved into the main branch pretty soon (or immediately). I'm
pretty
> confident that this new stuff will work as well as the main tree - just
need
> a couple of others to try it out, I guess.
>
> In short:
> - JDBCMailRepository (and Spool) now use Avalon/Cornerstone connection
> pooling and configuration. Just set one dbUrl for the whole app, finally!
> - Moved all of the SQL statements for the Mail repository into a single
file
> (sqlResources.xml), along with the SQL for the Users repository
> - Removed the various dot.properties files for the Mail repository, and
> moved all of the configuration into the james-config file.
> - Fixed the JDBCMailRepository to work for user Inbox.
> - Changed UsersJdbcRepository config to use a "destinationURL" syntax.
>
> Please see the README for more details.
>
> To build:
> .../build.sh -buildfile proposals/userstore/build.xml
>
> NOTE: You need to run this as "dist/bin/run.sh", since paths are currently
> set relative to the current directory. Will need to fix this to use the
> BlockContext path.
>
> enjoy
> Daz
---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]
