[Kde-pim] Review Request 112774: Support SILENT parameter in STORE command

[Christian Mollekopf]

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


> I'm not 100% whether we can actually do it this way - there's a risk that clients could overwrite the change (?), but I don't see any other way how to do it right now :)

I don't think there is any way around that if you want to avoid the notifications altogether. For the case of GID migration I don't think that this is a problem as one has to explicitly update the GID (so it's unlikely to be owerwritten, and if it is we don't care). The second issue are outdated caches that are again unlikely to be a problem in case of GID.
Do you have other usecases in mind that could help identify some problems (I can't)?

- Christian Mollekopf


On Sept. 17, 2013, 12:55 p.m., Dan Vrátil wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/112774/
> -----------------------------------------------------------
> 
> (Updated Sept. 17, 2013, 12:55 p.m.)
> 
> 
> Review request for Akonadi and Volker Krause.
> 
> 
> Description
> -------
> 
> Server counterpart to https://git.reviewboard.kde.org/r/112773/
> 
> I'm not 100% whether we can actually do it this way - there's a risk that clients could overwrite the change (?), but I don't see any other way how to do it right now :)
> 
> 
> Diffs
> -----
> 
>   server/src/handler/store.h a9a31b7 
>   server/src/handler/store.cpp b87cfe3 
>   server/src/storage/datastore.h 85a46b4 
>   server/src/storage/datastore.cpp 708e2b3 
> 
> Diff: http://git.reviewboard.kde.org/r/112774/diff/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Dan Vrátil
> 
>

_______________________________________________
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/