[ https://issues.apache.org/jira/browse/LUCENENET-106?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12665719#action_12665719 ]
eyalp edited comment on LUCENENET-106 at 1/20/09 10:24 PM: --------------------------------------------------------------- The fix for Digy's comment is very simple using a while loop: public bool MoveNext() { while (baseEnumerator.MoveNext()) { object key = ((WeakKey)baseEnumerator.Key).Target; if (key != null) { this.currentKey = key; this.currentValue = baseEnumerator.Value; return true; } } return false; } The problem with relying on Keys for enumeration is that calling Keys creates a temporary copy of all keys (which is slow). I'm adding comments to my last version and then i'll post it for inclusion. was (Author: eyalp): The fix for Digy's comment is very simple using a while loop: public bool MoveNext() { while (baseEnumerator.MoveNext()) { object key = ((WeakKey)baseEnumerator.Key).Target; if (key != null) { this.currentKey = key; this.currentValue = baseEnumerator.Value; return true; } } return false; } The problem with relying on Keys for enumeration is that calling Keys creates a temporary copy of all keys (which is slow). > Lucene.NET (Revision: 603121) is leaking memory > ----------------------------------------------- > > Key: LUCENENET-106 > URL: https://issues.apache.org/jira/browse/LUCENENET-106 > Project: Lucene.Net > Issue Type: Bug > Environment: .NET 2.0 > Reporter: Anton K. > Assignee: Digy > Priority: Critical > Attachments: DIGY-FieldCacheImpl.patch, Digy.rar, > luceneSrc_memUsage.patch, Paches for v2.3.1.rar, WeakHashTable v2.patch, > WeakHashTable v2.patch, WeakHashTable+FieldCacheImpl.rar, > WeakHashTable_ep.zip, WeakHashTable_ep_v2.zip, WeakHashTable_tj.zip, > WeakReferences.rar > > > readerCache Hashtable field (see FieldCacheImpl.cs) never releases some hash > items that have closed IndexReader object as a key. So a lot of Term > instances are never released. > Java version of Lucene uses WeakHashMap and therefore doesn't have this > problem. > This bug can be reproduced only when Sort functionality used during search. > See following link for additional information. > http://www.gossamer-threads.com/lists/lucene/java-user/55681 > Steps to reproduce: > 1)Create index > 2) Modify index by IndexWiter; Close IndexWriter > 3) Use IndexSearcher for searching with Sort; Close InexSearcher > 4) Go to step 2 > You'll get OutOfMemoryException after some time of running this algorithm. -- This message is automatically generated by JIRA. - You can reply to this email to add a comment to the issue online.