lucene-java-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From markrmil...@apache.org
Subject svn commit: r825721 - /lucene/java/branches/flex_1458/src/java/org/apache/lucene/index/codecs/standard/StandardTermsDictReader.java
Date Fri, 16 Oct 2009 01:00:47 GMT
Author: markrmiller
Date: Fri Oct 16 01:00:47 2009
New Revision: 825721

URL: http://svn.apache.org/viewvc?rev=825721&view=rev
Log:
remove a bunch of stuff I didn't mean to commit - must have left it in when I merged from
trunk - sorry - was just experimenting trying to figure out why a test was running out of
RAM

Modified:
    lucene/java/branches/flex_1458/src/java/org/apache/lucene/index/codecs/standard/StandardTermsDictReader.java

Modified: lucene/java/branches/flex_1458/src/java/org/apache/lucene/index/codecs/standard/StandardTermsDictReader.java
URL: http://svn.apache.org/viewvc/lucene/java/branches/flex_1458/src/java/org/apache/lucene/index/codecs/standard/StandardTermsDictReader.java?rev=825721&r1=825720&r2=825721&view=diff
==============================================================================
--- lucene/java/branches/flex_1458/src/java/org/apache/lucene/index/codecs/standard/StandardTermsDictReader.java
(original)
+++ lucene/java/branches/flex_1458/src/java/org/apache/lucene/index/codecs/standard/StandardTermsDictReader.java
Fri Oct 16 01:00:47 2009
@@ -188,9 +188,6 @@
   
   private class FieldReader extends Terms {
     private final CloseableThreadLocal threadResources = new CloseableThreadLocal();
-    // nocommit: not needed?
-    // nocommit: check placement
-    //Collection<ThreadResources> threadResourceSet = new HashSet<ThreadResources>();
     final long numTerms;
     final FieldInfo fieldInfo;
     final long termsStartPointer;
@@ -215,12 +212,6 @@
 
     public void close() {
       threadResources.close();
-      // nocommit should not be needed?
-      /*
-      for(ThreadResources threadResource : threadResourceSet) {
-        threadResource.termInfoCache = null;
-      }
-      */
     }
     
     private ThreadResources getThreadResources() throws IOException {
@@ -532,23 +523,6 @@
       } 
       return remove;
     }
-
-    // nocommit -- not needed?  we don't need to sync since
-    // only one thread works with this?
-
-    /*
-    @Override
-    public synchronized Object put(Object key, Object value) {
-      // TODO Auto-generated method stub
-      return super.put(key, value);
-    }
-    
-    @Override
-    public synchronized Object get(Object key) {
-      // TODO Auto-generated method stub
-      return super.get(key);
-    }
-    */
   }
 
 }



Mime
View raw message