From: Brett Porter <brett@apache.org>
Date: Thu, 8 Jun 2006 04:35:49 +0000 (+0000)
Subject: [MRM-118] clean up comments after some investigation
X-Git-Tag: archiva-0.9-alpha-1~834
X-Git-Url: https://source.dussan.org/?a=commitdiff_plain;h=f3a4d92ba5ed6ab48aca386f7c4a329c40811151;p=archiva.git

[MRM-118] clean up comments after some investigation

git-svn-id: https://svn.apache.org/repos/asf/maven/repository-manager/trunk@412651 13f79535-47bb-0310-9956-ffa450edef68
---

diff --git a/maven-repository-indexer/src/main/java/org/apache/maven/repository/indexing/AbstractRepositoryIndex.java b/maven-repository-indexer/src/main/java/org/apache/maven/repository/indexing/AbstractRepositoryIndex.java
index 5617dc85d..f65c6ff9d 100644
--- a/maven-repository-indexer/src/main/java/org/apache/maven/repository/indexing/AbstractRepositoryIndex.java
+++ b/maven-repository-indexer/src/main/java/org/apache/maven/repository/indexing/AbstractRepositoryIndex.java
@@ -35,21 +35,21 @@ import java.util.zip.ZipEntry;
  * Abstract class for RepositoryIndexers.
  *
  * @author Edwin Punzalan
+ * @todo [BP] overall am not happy with the design of this class and subclasses, but will refactor over time based on how it is used and by assessing how this affects Lucene's performance
  */
 public abstract class AbstractRepositoryIndex
     implements RepositoryIndex
 {
-    // TODO [!] can this be derived from the repository?
+    // TODO: can this be derived from the repository? -- probably a sensible default, but still should be configurable, but this could just be on the call to open()
     private String indexPath;
 
     private boolean indexOpen;
 
-    // TODO [!] why is the writer open for the life, but not the reader? why keep them open that length of time anyway? investigate best practices in Lucene
+    // TODO: why is the writer open for the life, but not the reader? why keep them open that length of time anyway? investigate best practices in Lucene
     private IndexWriter indexWriter;
 
     protected ArtifactRepository repository;
 
-    // TODO [!] is this really needed externally?
     private Analyzer analyzer;
 
     /**
@@ -158,7 +158,7 @@ public abstract class AbstractRepositoryIndex
     protected IndexWriter getIndexWriter()
         throws IOException
     {
-        // TODO [!] why is this allowed to be called before open()?
+        // TODO: why is this allowed to be called before open()?
         if ( indexWriter == null )
         {
             indexWriter = new IndexWriter( indexPath, getAnalyzer(), false );