[Bug 33213] Rename _get_events() to get_entities()

bugzilla-daemon at freedesktop.org bugzilla-daemon at freedesktop.org
Thu Jan 20 15:22:49 CET 2011


--- Comment #6 from Nicolas Dufresne <nicolas.dufresne at collabora.co.uk> 2011-01-20 06:22:48 PST ---
(In reply to comment #4)
> > Note that getting all events for an
> > account is a bad idea. If they want to do like the N900 we will implement a two
> > way iterator or something similar. Do you have a draft for this proposal ?
> See https://bugzilla.gnome.org/show_bug.cgi?id=619866
> Why is it a bad idea? It may be too much? If so it would be fine to just get
> the last N events, with a possibility to get more and more in subsequent
> queries. Also having everything in sqlite would make this less resource-hungry.

Thanks for the link, I'll have a close look. Having a get_events() function
that return all events for an account will result in degrading performance over
years. While the data itself is probably not so big, it will consume a lot of
memory when put inside the widget. Progressive loading (probably based on get
next N, get prev N) would ensure that performance will not degrade and reduce
memory footprint. Let's discuss this somewhere else though.

Configure bugmail: https://bugs.freedesktop.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the QA contact for the bug.
You are the assignee for the bug.

More information about the telepathy-bugs mailing list