Adding a CouchDB backend
Sebastian Spaeth
Sebastian at SSpaeth.de
Mon Sep 5 15:17:01 BST 2011
On Sat, 3 Sep 2011 17:09:58 +0200, "=?utf-8?B?IFRow6lvZG9yZSBCaWFkYWxh?=" <theodore at biadala.net> wrote:
Cool, a couple of code comments:
https://github.com/theodoreb/offlineimap/blob/couchdb-lite/offlineimap/folder/CouchDB.py
__init__ looks fine as it is.
parseToCouch():
I see that you store them with the Message-ID as key. In theory (and
also in practice) you can have 2 emails with the exact same Message
ID. In this case, you overwrite your messages.
msgid = msgid[1:-1] Would be nice to have a comment that you are
cutting of pointy brackets here.
The rest of the file actually looks quite ok to me. I see a couple of
things we should migrate to the Base folder implementation, like the
definition of getaccountname().
> https://github.com/theodoreb/offlineimap/blob/couchdb-lite/offlineimap/repository/CouchDB.py
Looking good to me. Even better if it works.
> The config file looks like this :
>
> [Repository Couch]
> type = CouchDB
> dbname = offlineimap
> server = http://localhost:5984/
>
> Works as local and remote repository.
Nice. Seems decent and optional enough to push as experimental. My only
reservation is the reliance on unique Message-IDs for the filenames.
Sebastian
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 197 bytes
Desc: not available
URL: <http://alioth-lists.debian.net/pipermail/offlineimap-project/attachments/20110905/a4fdf19b/attachment-0001.sig>
More information about the OfflineIMAP-project
mailing list