Improving logging statements

diff --git a/archiva-modules/plugins/maven2-repository/src/main/java/org/apache/archiva/dependency/tree/maven2/ArchivaRepositoryConnectorFactory.java b/archiva-modules/plugins/maven2-repository/src/main/java/org/apache/archiva/dependency/tree/maven2/ArchivaRepositoryConnectorFactory.java
index 4ce243f98e896562dcbb96c961fa44a52870fa80..dd401aef482ece4565d1aaa3bdd210680da29559 100644 (file)
--- a/archiva-modules/plugins/maven2-repository/src/main/java/org/apache/archiva/dependency/tree/maven2/ArchivaRepositoryConnectorFactory.java
+++ b/archiva-modules/plugins/maven2-repository/src/main/java/org/apache/archiva/dependency/tree/maven2/ArchivaRepositoryConnectorFactory.java
@@ -34,6 +34,9 @@ import org.eclipse.aether.transfer.NoRepositoryConnectorException;
 import java.util.Collection;
 
 /**
+ *
+ * Creates a dummy connector, if the default connectory factory fails to create one.
+ *
  * @author Olivier Lamy
  * @since 1.4-M3
  */

diff --git a/archiva-modules/plugins/maven2-repository/src/main/java/org/apache/archiva/repository/maven2/MavenUtil.java b/archiva-modules/plugins/maven2-repository/src/main/java/org/apache/archiva/repository/maven2/MavenUtil.java
index f9e69291ef613ccc2c50a5248a90d374c1a09dff..95d4df019100f2f8feac053a5164094532d32a23 100644 (file)
--- a/archiva-modules/plugins/maven2-repository/src/main/java/org/apache/archiva/repository/maven2/MavenUtil.java
+++ b/archiva-modules/plugins/maven2-repository/src/main/java/org/apache/archiva/repository/maven2/MavenUtil.java
@@ -39,9 +39,23 @@ import org.eclipse.aether.repository.NoLocalRepositoryManagerException;
 import org.eclipse.aether.spi.connector.RepositoryConnectorFactory;
 import org.eclipse.aether.util.graph.selector.AndDependencySelector;
 import org.eclipse.aether.util.graph.selector.ExclusionDependencySelector;
+import org.slf4j.Logger;
+import org.slf4j.LoggerFactory;
 
+/**
+ * Some static utility methods that are used by different classes.
+ */
 public class MavenUtil {
 
+    static Logger log = LoggerFactory.getLogger(MavenUtil.class);
+
+    /**
+     * Creates a new aether repository system session for the given directory and assigns the
+     * repository to this session.
+     *
+     * @param localRepoDir The repository directory
+     * @return The newly created session object.
+     */
     public static RepositorySystemSession newRepositorySystemSession(String localRepoDir)
     {
         DefaultRepositorySystemSession session = MavenRepositorySystemUtils.newSession( );
@@ -58,12 +72,16 @@ public class MavenUtil {
         }
         catch ( NoLocalRepositoryManagerException e )
         {
-            e.printStackTrace( );
+            log.error("Could not assign the repository manager to the session: {}", e.getMessage(), e);
         }
 
         return session;
     }
 
+    /**
+     * Finds the
+     * @return
+     */
     public static RepositorySystem newRepositorySystem()
     {
         DefaultServiceLocator locator = MavenRepositorySystemUtils.newServiceLocator( );