Review Request 110739: Create and cache the contact groups

[Vishesh Handa]

> On May 31, 2013, 5:42 p.m., David Edmundson wrote:
> > There is a hypothetical issue that if the group were to be deleted by an external source the cache is now pointing to URIs that don't exist. Do you see this as something that would be a problem?

Not really.

The only case that can happen is when a developer does that, and in that case, he/she should restart Nepomuk. Or if some user decides to delete his Nepomuk repository. Again, they should restart Nepomuk.


- Vishesh


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


On May 30, 2013, 6:47 p.m., Vishesh Handa wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/110739/
> -----------------------------------------------------------
> 
> (Updated May 30, 2013, 6:47 p.m.)
> 
> 
> Review request for Telepathy, Aleix Pol Gonzalez, David Edmundson, and Martin Klapetek.
> 
> 
> Description
> -------
> 
>     Create and cache the contact groups
>     
>     Earlier, if a group contained more than one contact then it the contact
>     group was created that many times cause a new SimpleResource was created
>     each time.
>     
>     With this method we cache the contact groups so that they are only
>     created once, and once they have been created we do not unnecessarily
>     send a new SimpleResource to get merged into the existing contact group.
> 
> 
> Diffs
> -----
> 
>   nepomuk-storage.h ad5b6bc 
>   nepomuk-storage.cpp adee609 
> 
> Diff: http://git.reviewboard.kde.org/r/110739/diff/
> 
> 
> Testing
> -------
> 
> No more duplicate contact groups.
> 
> 
> Thanks,
> 
> Vishesh Handa
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-telepathy/attachments/20130603/38b4fe25/attachment.html>