[Kde-pim] Akonadi Synchronous Job Error

[Jason 'vanRijn' Kasper]

On Sat, Feb 21, 2009 at 9:31 AM, Volker Krause <vkrause at kde.org> wrote:

> On Friday 20 February 2009 17:04:20 Jason 'vanRijn' Kasper wrote:
> > On Fri, Feb 20, 2009 at 7:23 AM, Volker Krause <vkrause at kde.org> wrote:
> > > On Friday 20 February 2009 13:02:57 Douglas Harms wrote:
> > > > I'm working on kpilot and have encountered a behavior problem in
> > > > akonadi.  For various reasons there are times when we attempt to
> delete
> > > > an akonadi item (calendar or todo) that doesn't exist.  job->exec()
> > > > returns false (as it should).  When we process the next record (e.g.,
> > > > delete a record that is in akonadi) job->exec() returns true (which
> it
> > > > should) but it doesn't actually delete the item.  As you might
> imagine,
> > > > this messes us up.
> > > >
> > > > We can easily solve this by not trying to delete an akonadi item when
> > > > we know it's not there; however, we wanted to bring this up for
> > > > discussion on this list to see if perhaps there is a larger bug in
> > > > akonadi that should be addressed.
> > >
> > > Are you deleting within a transaction maybe? If there is an error
> within
> > > a transaction, no change made within it will persist, like in database
> > > systems.
> > > If that should happen for usually harmless errors like deleting a
> > > non-existing
> > > object is a different question though
> >
> > I'm not honestly sure if what we're doing is within a transaction.I don't
> > see anywhere that we're explicitly doing a
> > beginTransaction()/endTransaction() or something simliar. We have commit
> > methods in kpilot for each type of operation (create, update, delete) to
> > push the temp changes we made during synchronization into Akonadi, and
> the
> > code for each looks like this:
>
> turns out this was indeed caused by a mis-use of transactions, but in the
> Akonadi code itself. Should be fixed in trunk now, I'll backport in a
> minute.


Thanks, Volker!


-- 
-[ Jason 'vanRijn' Kasper    //  http://movingparts.net ]-
-[ KDE PIM Developer         //  http://www.kde.org  ]-
-[ bash fun -> :(){ :|:&};:  //  Numbers 6:22-26 ]-
_______________________________________________
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/