http://www.slony.info/bugzilla/show_bug.cgi?id=321

--- Comment #3 from Steve Singer <ssinger at ca.afilias.info> 2013-10-18 10:27:46 PDT ---
I *THINK* this change is breaking the Failover unit test.

I am occasionally getting a failure in the last part of the failover test where
db4 is missing data.  It appears to be receiving a SYNC event from 1 (the
failed node) via a event provider that isn't in the provider list.  It then
ignores the SYNC (marks it as confirmed without actually pulling data for that
SYNC).

This happens maybe once every 4-6 times I run through the failover node, on the
last test (the one designed to detect bug 318).

I have not yet confirmed that this problem is caused by this patch, I still
have to run the test more times without 321 applied to build confidence in
this.

I did however replace the adjust_provider call this patch removes with a 
slon_log(SLON_WARN, ".... event provider %d is not in the provider list")

and I am getting messages like 
 WARN   remoteWorkerThread_1: event provider 5 is not in the provider list

when processing SYNC's from the failed node on db4.

-- 
Configure bugmail: http://www.slony.info/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug.