~robru/friends/0.1.2-0ubuntu1

Viewing all changes in revision 157.

  • Committer: Robert Bruce Park
  • Date: 2013-03-04 18:18:39 UTC
  • mfrom: (156.1.1 trunk)
  • Revision ID: robert.park@canonical.com-20130304181839-iiurmnswma6l4red
Bring up DBus interface sooner during startup.

We were having some racy timeout issues where we'd be invoked by dbus,
but then didn't get the dispatcher up in time to respond to dbus
method invocations before dbus timed out on us, because waiting for
the DeeModel to sync was too slow.

Instead, we bring up the dispatcher first, but also acquire the
_publish_lock, which prevents the dispatcher from trying to publish
prematurely (eg, before caches have been populated and and the
DeeModel syncronized). Then once the DeeModel syncs, we release the
lock and allow the dispatcher to function as normal.

Ken & I both tested this branch throughout the entire weekend and not
a single timeout occurred the entire time, so this is a big win.

expand all expand all

Show diffs side-by-side

added added

removed removed

Lines of Context: