[Kde-pim] Review Request: Store payload data in files

[Volker Krause]

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://reviewboard.kde.org/r/13/#review30
-----------------------------------------------------------

Ship it!


Yes, please go ahead and commit.

- Volker


On 2009-02-05 10:56:20, Andras Mantia wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://reviewboard.kde.org/r/13/
> -----------------------------------------------------------
> 
> (Updated 2009-02-05 10:56:20)
> 
> 
> Review request for KDE PIM.
> 
> 
> Summary
> -------
> 
> Preliminary implementation of storing the payload data in files instead of the database. It work for me, but it is not well tested. I'd like to get a general feedback about the idea how it is implemented (or anything else, if you have a comment). The basic idea is to use a helper class when dealing with Part objects, and the helper class decides where to store/load the data.
> 
> 
> Diffs
> -----
> 
>   trunk/kdesupport/akonadi/server/src/handler/store.cpp 921820 
>   trunk/kdesupport/akonadi/server/src/handler/copy.cpp 921820 
>   trunk/kdesupport/akonadi/server/src/handler/fetch.cpp 921820 
>   trunk/kdesupport/akonadi/server/src/handler/append.cpp 921820 
>   trunk/kdesupport/akonadi/server/CMakeLists.txt 921820 
>   trunk/kdesupport/akonadi/server/src/akonadi.cpp 921820 
>   trunk/kdesupport/akonadi/server/src/akonadiconnection.h 921820 
>   trunk/kdesupport/akonadi/server/src/cachecleaner.cpp 921820 
>   trunk/kdesupport/akonadi/server/src/global.h 921820 
>   trunk/kdesupport/akonadi/server/src/storage/akonadidb.xml 921820 
>   trunk/kdesupport/akonadi/server/src/storage/datastore.cpp 921820 
>   trunk/kdesupport/akonadi/server/src/storage/entities-header.xsl 921820 
>   trunk/kdesupport/akonadi/server/src/storage/parthelper.h PRE-CREATION 
>   trunk/kdesupport/akonadi/server/src/storage/parthelper.cpp PRE-CREATION 
> 
> Diff: http://reviewboard.kde.org/r/13/diff
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Andras
> 
>

_______________________________________________
KDE PIM mailing list kde-pim at kde.org
https://mail.kde.org/mailman/listinfo/kde-pim
KDE PIM home page at http://pim.kde.org/