Memory Leak because of ClosableThreadLocal

2010-12-15 Thread Ayende Rahien
The reasoning for this class doesn't hold for .NET, but that is beside the point. There is somewhere in Lucene where this doesn't clean up (specifically, ThreadResources for Term Cache) That results in ClosableThreadLocal.slots filling up. I fixed this by modifying ClosableThreadLocal to use the .

Re: Memory Leak because of ClosableThreadLocal

2010-12-15 Thread Robert Jordan
On 15.12.2010 09:12, Ayende Rahien wrote: The reasoning for this class doesn't hold for .NET, but that is beside the point. There is somewhere in Lucene where this doesn't clean up (specifically, ThreadResources for Term Cache) That results in ClosableThreadLocal.slots filling up. I fixed this b

Re: Memory Leak because of ClosableThreadLocal

2010-12-15 Thread Ayende Rahien
No, I am using Lucene.NET 2.9.2, and it showed up there. More specifically, I forked it around r48322 On Wed, Dec 15, 2010 at 2:02 PM, Robert Jordan wrote: > On 15.12.2010 12:58, Robert Jordan wrote: > >> On 15.12.2010 09:12, Ayende Rahien wrote: >> >>> The reasoning for this class doesn't hold

Re: Memory Leak because of ClosableThreadLocal

2010-12-15 Thread Robert Jordan
On 15.12.2010 13:58, Ayende Rahien wrote: No, I am using Lucene.NET 2.9.2, and it showed up there. More specifically, I forked it around r48322 I have no idea where you got "r48322" from. The class was reworked in r942795 and r947759. See LUCENENET-358. Robert On Wed, Dec 15, 2010 at 2:02