search

PearsonEducation / thalassa

Other
76 stars 9 forks source link

Fix for Network Partition Race Condition #4

Closed RBeaudoin closed 10 years ago

RBeaudoin commented 10 years ago

Update the registry to always emit 'online' events when a client checks in. This should address the edge case where there is a network partition and the initial 'online' event is missed due to network issues.

The client has been updated to maintain an in-memory collection of all registrations related to it's current subscriptions. The client will only emit an 'online' event if the registration metadata has changed.

rem-aj commented 10 years ago

Hi @RBeaudoin

So this fix has been pushed out to all the thallassa servers in VPC2?

Thanks!

RBeaudoin commented 10 years ago

@bostonmoto no, this pull request has to be merged. The issue will remain open until that point in time.

rem-aj commented 10 years ago

Thanks Mike & Rich. Any ETA on when this fix will be applied to to thalassa servers in VPC2?

On Mon, Apr 14, 2014 at 6:38 PM, Mike Brevoort notifications@github.comwrote:

Merged #4 https://github.com/PearsonEducation/thalassa/pull/4.

— Reply to this email directly or view it on GitHubhttps://github.com/PearsonEducation/thalassa/pull/4 .