Descriptionstate/api/watcher: Client-side watchers improved
This branch fixes an issue with StringsWatcher at
client-side: it was possible to drop events if you
do not keep reading constantly. Now they get queued
up and arrive in the order delivered when reading.
Added a test as well.
Also, as a drive-by fix: there was a possibility for
deadlocking in both StringsWatcher and NotifyWatcher
when trying to close it while there is an event waiting
to be read. Added tests for these cases as well and
verified that without the fix deadlocks are indeed
triggered.
https://code.launchpad.net/~dimitern/juju-core/071-stringswatcher-client-improvements/+merge/175485
(do not edit description out of merge proposal)
Patch Set 1 #
Total comments: 12
Patch Set 2 : state/api/watcher: StringsWatcher improvements #
Total comments: 2
Patch Set 3 : state/api/watcher: StringsWatcher improvements #
Total comments: 14
Patch Set 4 : state/api/watcher: Client-side watchers improved #
MessagesTotal messages: 8
|