thanks for your super fast code review.
I've changed the method name and fixed some engine-testcases (Exceptions are
not raised anymore, 0 is returned instead)
If nobody complains until lunchtime I will merge this branch into lp:zeitgeist.
This branch fixes only half of the issue, we also need a way to handle
GetEvents, which seems abit more complicated to me, but is a different bug,
which I will file in a bit.
What to do on failed or blocked event insertions?
You received this bug notification because you are a member of Zeitgeist
Framework, which is the registrant for Zeitgeist Framework.
Status in Zeitgeist Framework: Confirmed
This bug is filed in relation to bug #495017: "AttributeError: 'NoneType'
object has no attribute 'payload'"
Markus asks what to do with failed or blocked events in InsertEvents(). This is
an important questions since we can't not raise an error or simply leave the
event id out of the response. Clients depend on mapping the returned
I propose to say that event id 0, indicates an error. I think this is a nice
solution: easy to implement without breaking API, and easy to handle for
clients because a boolean test "if not event_id : print 'eeeeeek!'" works.
Mailing list: https://launchpad.net/~zeitgeist
Post to : email@example.com
Unsubscribe : https://launchpad.net/~zeitgeist
More help : https://help.launchpad.net/ListHelp