Attached a patch implementing my proposal.

Regards,
Andreas

Andreas Pflug wrote:
> We've been discussing using nodes/paths to store information for 
> non-slon active node, but didn't come to a conclusion so far.
> 
> Currently, listens will be generated for every path regardless of the 
> node's activity. If a node will never have a running slon daemon, events 
> for this node will collect and prevent cleanup of old events, which is bad.
> 
> As a fix, I'd propose to generate listens from paths only between 
> _active_ nodes, i.e. add "and N1.no_active and N2.no_active" to the 
> select condition in rebuildlistenentries(). A node that's not active 
> will never process events, and never generate events so IMHO it's 
> correct to have no listens for such nodes. In addition, 
> rebuildlistenentries() probably needs to be added to enablenode() and 
> disablenode() (as soon as it's implemented) too.
> 
> Regards,
> Andreas
> _______________________________________________
> Slony1-general mailing list
> Slony1-general at gborg.postgresql.org
> http://gborg.postgresql.org/mailman/listinfo/slony1-general

-------------- next part --------------
A non-text attachment was scrubbed...
Name: rebuildListenEntries.patch
Type: text/x-patch
Size: 833 bytes
Desc: not available
Url : http://gborg.postgresql.org/pipermail/slony1-general/attachments/20050925/761ac75c/rebuildListenEntries.bin