zeitgeist team mailing list archive

[Mikkel Kamstrup Erlandsen <mikkel.kamstrup@xxxxxxxxx>]

We could use negative integers as error indicators, but that would
require that we break DBus API since we use uint32 (signatire u) as
event _ids now. The alternative is to use 64 bit ints (signature x).
It's not a big break though - transpararent in Python actually...

Looking at your branch now.

-- 
What to do on failed or blocked event insertions?
https://bugs.launchpad.net/bugs/495179
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

Bug description:
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 event_ids->events.

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.