ItemSync can be canceled but not defered. Add assert and debug output.
ClosedPublic

Authored by dfaure on Aug 22 2018, 7:42 AM.

Details

Summary

And turn the warning in rollback to a debug, this isn't a problem when
it happens because the task is canceled (e.g. interval-sync with no network
available).

Test Plan

I've been running with this for some time and I never hit
the assert (after fixing the imap resource and the FatCRM resources to do this
correctly). Let's hope no other resource hits this. But OTOH, if they were deferring
an itemsync, they would get stuck forever, the job never finishing...

Diff Detail

Repository
R165 Akonadi
Branch
2018_08_defer_assert
Lint
No Linters Available
Unit
No Unit Test Coverage
Build Status
Buildable 2194
Build 2212: arc lint + arc unit
dfaure created this revision.Aug 22 2018, 7:42 AM
Restricted Application added a project: KDE PIM. · View Herald TranscriptAug 22 2018, 7:42 AM
Restricted Application edited subscribers, added: kde-pim; removed: KDE PIM. · View Herald Transcript
dfaure requested review of this revision.Aug 22 2018, 7:42 AM
dfaure updated this revision to Diff 40353.Aug 24 2018, 8:52 AM

Use qCDebug

dvratil accepted this revision.Aug 24 2018, 8:18 PM
This revision is now accepted and ready to land.Aug 24 2018, 8:18 PM
This revision was automatically updated to reflect the committed changes.