Updated some javadocs

git-svn-id: https://svn.apache.org/repos/asf/maven/repository-manager/trunk@414549 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 5903cebb69b2b79f80f882acd943f31b88bde1be..30ea608b2b10ea35052c52394f45281edb0fe3fc 100644 (file)
--- 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
@@ -200,6 +200,9 @@ public abstract class AbstractRepositoryIndex
         deleteDocuments( Collections.singletonList( term ) );
     }
 
+    /**
+     * @see RepositoryIndex#deleteDocuments(java.util.List) 
+     */
     public void deleteDocuments( List termList )
         throws RepositoryIndexException, IOException
     {
@@ -228,11 +231,7 @@ public abstract class AbstractRepositoryIndex
     }
 
     /**
-     * Opens the lucene index and add all the lucene documents inside the list into the index.
-     * Closes the index at the end.
-     *
-     * @param docList List of Lucene Documents
-     * @throws RepositoryIndexException when an error occurred during the indexing of the documents
+     * @see RepositoryIndex#addDocuments(java.util.List)
      */
     public void addDocuments( List docList )
         throws RepositoryIndexException

diff --git a/maven-repository-indexer/src/main/java/org/apache/maven/repository/indexing/ArtifactRepositoryIndex.java b/maven-repository-indexer/src/main/java/org/apache/maven/repository/indexing/ArtifactRepositoryIndex.java
index 9d669af64d54b2c6084317c115914bf64502bc90..8a1f8fb553348546154b04554c5c24a8cfbedc30 100644 (file)
--- a/maven-repository-indexer/src/main/java/org/apache/maven/repository/indexing/ArtifactRepositoryIndex.java
+++ b/maven-repository-indexer/src/main/java/org/apache/maven/repository/indexing/ArtifactRepositoryIndex.java
@@ -62,24 +62,12 @@ public class ArtifactRepositoryIndex
     }
 
     /**
-     * Checks if the artifact has already been indexed and deletes it if it is.
+     * Indexes the artifacts found within the specified list.  Deletes existing indices for the same artifacts first,
+     * before proceeding on adding them into the index.
      *
-     * @param artifact the object to be indexed.
+     * @param artifactList
      * @throws RepositoryIndexException
      */
-    private void deleteIfIndexed( Artifact artifact )
-        throws RepositoryIndexException
-    {
-        try
-        {
-            deleteDocument( FLD_ID, ARTIFACT + ":" + artifact.getId() );
-        }
-        catch ( IOException e )
-        {
-            throw new RepositoryIndexException( "Failed to delete a document", e );
-        }
-    }
-
     public void indexArtifacts( List artifactList )
         throws RepositoryIndexException
     {
@@ -95,6 +83,12 @@ public class ArtifactRepositoryIndex
         addDocuments( getDocumentList( artifactList ) );
     }
 
+    /**
+     * Creates a list of Lucene Term object used in index deletion
+     *
+     * @param artifactList
+     * @return List of Term object
+     */
     private List getTermList( List artifactList )
     {
         List list = new ArrayList();
@@ -109,6 +103,13 @@ public class ArtifactRepositoryIndex
         return list;
     }
 
+    /**
+     * Creates a list of Lucene documents, used for index additions
+     *
+     * @param artifactList
+     * @return
+     * @throws RepositoryIndexException
+     */
     private List getDocumentList( List artifactList )
         throws RepositoryIndexException
     {
@@ -136,6 +137,13 @@ public class ArtifactRepositoryIndex
         indexArtifacts( Collections.singletonList( artifact ) );
     }
 
+    /**
+     * Creates a Lucene Document from an artifact; used for index additions
+     *
+     * @param artifact
+     * @return
+     * @throws RepositoryIndexException
+     */
     private Document createDocument( Artifact artifact )
         throws RepositoryIndexException
     {

diff --git a/maven-repository-indexer/src/main/java/org/apache/maven/repository/indexing/EclipseRepositoryIndex.java b/maven-repository-indexer/src/main/java/org/apache/maven/repository/indexing/EclipseRepositoryIndex.java
index 52795d84ccdae52acee4132b99fafb5fdfbb91ea..a0a298c84428e4f7dadcfdec1cc1a9d5920edae1 100644 (file)
--- a/maven-repository-indexer/src/main/java/org/apache/maven/repository/indexing/EclipseRepositoryIndex.java
+++ b/maven-repository-indexer/src/main/java/org/apache/maven/repository/indexing/EclipseRepositoryIndex.java
@@ -88,6 +88,12 @@ public class EclipseRepositoryIndex
         return new EclipseIndexAnalyzer( new SimpleAnalyzer() );
     }
 
+    /**
+     * Indexes the artifacts inside the provided list
+     *
+     * @param artifactList
+     * @throws RepositoryIndexException
+     */
     public void indexArtifacts( List artifactList )
         throws RepositoryIndexException
     {
@@ -124,6 +130,13 @@ public class EclipseRepositoryIndex
         }
     }
 
+    /**
+     * Creates a Lucene Document from an artifact; used for index additions
+     *
+     * @param artifact
+     * @return
+     * @throws RepositoryIndexException
+     */
     private Document createDocument( Artifact artifact )
         throws RepositoryIndexException
     {

diff --git a/maven-repository-indexer/src/main/java/org/apache/maven/repository/indexing/MetadataRepositoryIndex.java b/maven-repository-indexer/src/main/java/org/apache/maven/repository/indexing/MetadataRepositoryIndex.java
index 6fad583adcba017b951dac62d0362e7b93d2df91..56b754188441eae23ae67fc012408317c36ab695 100644 (file)
--- a/maven-repository-indexer/src/main/java/org/apache/maven/repository/indexing/MetadataRepositoryIndex.java
+++ b/maven-repository-indexer/src/main/java/org/apache/maven/repository/indexing/MetadataRepositoryIndex.java
@@ -51,7 +51,7 @@ public class MetadataRepositoryIndex
     }\r
 \r
     /**\r
-     * Index the contents of the specified RepositoryMetadata paramter object\r
+     * Index the contents of the specified RepositoryMetadata parameter object\r
      *\r
      * @param repoMetadata the metadata object to be indexed\r
      * @throws RepositoryIndexException\r
@@ -62,6 +62,13 @@ public class MetadataRepositoryIndex
         indexMetadata( Collections.singletonList( repoMetadata ) );\r
     }\r
 \r
+    /**\r
+     * Index the metadata found within the provided list.  Deletes existing entries in the index first before\r
+     * proceeding with the index additions.\r
+     *\r
+     * @param metadataList\r
+     * @throws RepositoryIndexException\r
+     */\r
     public void indexMetadata( List metadataList )\r
         throws RepositoryIndexException\r
     {\r
@@ -77,6 +84,12 @@ public class MetadataRepositoryIndex
         addDocuments( getDocumentList( metadataList ) );\r
     }\r
 \r
+    /**\r
+     * Creates a list of Lucene Term object used in index deletion\r
+     *\r
+     * @param metadataList\r
+     * @return List of Term object\r
+     */\r
     private List getTermList( List metadataList )\r
     {\r
         List terms = new ArrayList();\r
@@ -91,6 +104,12 @@ public class MetadataRepositoryIndex
         return terms;\r
     }\r
 \r
+    /**\r
+     * Creates a list of Lucene documents\r
+     *\r
+     * @param metadataList\r
+     * @return List of Lucene Documents\r
+     */\r
     private List getDocumentList( List metadataList )\r
     {\r
         List docs = new ArrayList();\r
@@ -105,6 +124,12 @@ public class MetadataRepositoryIndex
         return docs;\r
     }\r
 \r
+    /**\r
+     * Creates a Lucene Document from a RepositoryMetadata; used for index additions\r
+     *\r
+     * @param repoMetadata\r
+     * @return Lucene Document\r
+     */\r
     private Document createDocument( RepositoryMetadata repoMetadata )\r
     {\r
         //get lastUpdated from Versioning (specified in Metadata object)\r
@@ -172,17 +197,4 @@ public class MetadataRepositoryIndex
         doc.add( Field.Keyword( FLD_PLUGINS_ALL, "" ) );\r
         return doc;\r
     }\r
-\r
-    private void deleteIfIndexed( RepositoryMetadata repoMetadata )\r
-        throws RepositoryIndexException\r
-    {\r
-        try\r
-        {\r
-            deleteDocument( FLD_ID, (String) repoMetadata.getKey() );\r
-        }\r
-        catch ( IOException e )\r
-        {\r
-            throw new RepositoryIndexException( "Failed to delete document", e );\r
-        }\r
-    }\r
 }\r

diff --git a/maven-repository-indexer/src/main/java/org/apache/maven/repository/indexing/PomRepositoryIndex.java b/maven-repository-indexer/src/main/java/org/apache/maven/repository/indexing/PomRepositoryIndex.java
index 7ca9f017918bc181fe123b79abf5bd42b07a7ef2..0a13c9716b104e890ee455e60709160c3fdf0491 100644 (file)
--- a/maven-repository-indexer/src/main/java/org/apache/maven/repository/indexing/PomRepositoryIndex.java
+++ b/maven-repository-indexer/src/main/java/org/apache/maven/repository/indexing/PomRepositoryIndex.java
@@ -68,19 +68,6 @@ public class PomRepositoryIndex
         this.artifactFactory = artifactFactory;
     }
 
-    private void deleteIfIndexed( Model pom )
-        throws RepositoryIndexException
-    {
-        try
-        {
-            deleteDocument( FLD_ID, POM + ":" + pom.getId() );
-        }
-        catch ( IOException e )
-        {
-            throw new RepositoryIndexException( "Failed to delete document", e  );
-        }
-    }
-
     /**
      * Method to create the index fields for a Model object into the index
      *
@@ -93,6 +80,12 @@ public class PomRepositoryIndex
         indexPoms( Collections.singletonList( pom ) );
     }
 
+    /**
+     * Index the Models within the supplied List.  Deletes existing index values before adding them to the list.
+     *
+     * @param pomList
+     * @throws RepositoryIndexException
+     */
     public void indexPoms( List pomList )
         throws RepositoryIndexException
     {
@@ -108,6 +101,12 @@ public class PomRepositoryIndex
         addDocuments( getDocumentList( pomList ) );
     }
 
+    /**
+     * Creates a list of Lucene Term object used in index deletion
+     *
+     * @param pomList
+     * @return List of Term object
+     */
     private List getTermList( List pomList )
     {
         List terms = new ArrayList();
@@ -122,6 +121,13 @@ public class PomRepositoryIndex
         return terms;
     }
 
+    /**
+     * Creates a list of Lucene documents
+     *
+     * @param pomList
+     * @return
+     * @throws RepositoryIndexException
+     */
     private List getDocumentList( List pomList )
         throws RepositoryIndexException
     {
@@ -137,6 +143,13 @@ public class PomRepositoryIndex
         return docs;
     }
 
+    /**
+     * Creates a Lucene Document from a Model; used for index additions
+     *
+     * @param pom
+     * @return
+     * @throws RepositoryIndexException
+     */
     private Document createDocument( Model pom )
         throws RepositoryIndexException
     {

diff --git a/maven-repository-indexer/src/main/java/org/apache/maven/repository/indexing/RepositoryIndex.java b/maven-repository-indexer/src/main/java/org/apache/maven/repository/indexing/RepositoryIndex.java
index ecbdfd6dbe905119858f1c34510c669f945ef0f9..8fb8ff73ef9957dcd3114af476ce5e65d2ff6506 100644 (file)
--- a/maven-repository-indexer/src/main/java/org/apache/maven/repository/indexing/RepositoryIndex.java
+++ b/maven-repository-indexer/src/main/java/org/apache/maven/repository/indexing/RepositoryIndex.java
@@ -120,9 +120,23 @@ public interface RepositoryIndex
     public void validate()
         throws RepositoryIndexException, IOException;
 
+    /**
+     * Opens the lucene index and add all the lucene documents inside the list into the index.
+     * Closes the index at the end.
+     *
+     * @param docList List of Lucene Documents
+     * @throws RepositoryIndexException when an error occurred during the indexing of the documents
+     */
     public void addDocuments( List docList )
         throws RepositoryIndexException;
 
+    /**
+     * Delete from the index matching the list of lucene Terms
+     *
+     * @param termList List of Lucene Term
+     * @throws RepositoryIndexException
+     * @throws IOException
+     */
     public void deleteDocuments( List termList )
         throws RepositoryIndexException, IOException;
 }