On 6/1/2010 9:31 AM, Steve Singer wrote: > Jan Wieck wrote: >> On 5/28/2010 6:57 PM, Steve Singer wrote: > >>> >>> (A version of 2.0.4 RC1 + these changes can be found at >>> http://github.com/ssinger/slony/tree/203_memfixes_UlrichWeber) >> >> Is that the memory leak from not freeing notifications? If so, let's >> test it with bombarding a slon with a gazillion notifies and if it >> survives, put it into 2.0.4. >> >> > > The patch frees memory in a number of places including the cleanup > thread and during the set copy. You can view the patch at the above URL. > > The 1.2 version of the patch also removed the notify stuff (but that > wasn't required with 2.0 the code was removed). If we are happy with > things we should apply it to 1.2 as well for the next time we do a 1.2.x > release.
After a detailed line by line review of the patch, Steve and I decided to commit it for 2.0.4. Jan -- Anyone who trades liberty for security deserves neither liberty nor security. -- Benjamin Franklin _______________________________________________ Slony1-general mailing list [email protected] http://lists.slony.info/mailman/listinfo/slony1-general
