svn commit: r396024 - in /geronimo/branches/1.1/modules/kernel/src/java/org/apache/geronimo/kernel: config/ConfigurationManager.java repository/ArtifactResolver.java

[am...@apache.org]

Author: ammulder
Date: Fri Apr 21 15:26:18 2006
New Revision: 396024

URL: http://svn.apache.org/viewcvs?rev=396024&view=rev
Log:
Add some docs

Modified:
    geronimo/branches/1.1/modules/kernel/src/java/org/apache/geronimo/kernel/config/ConfigurationManager.java
    geronimo/branches/1.1/modules/kernel/src/java/org/apache/geronimo/kernel/repository/ArtifactResolver.java

Modified: geronimo/branches/1.1/modules/kernel/src/java/org/apache/geronimo/kernel/config/ConfigurationManager.java
URL: http://svn.apache.org/viewcvs/geronimo/branches/1.1/modules/kernel/src/java/org/apache/geronimo/kernel/config/ConfigurationManager.java?rev=396024&r1=396023&r2=396024&view=diff
==============================================================================
--- geronimo/branches/1.1/modules/kernel/src/java/org/apache/geronimo/kernel/config/ConfigurationManager.java (original)
+++ geronimo/branches/1.1/modules/kernel/src/java/org/apache/geronimo/kernel/config/ConfigurationManager.java Fri Apr 21 15:26:18 2006
@@ -445,5 +445,10 @@
      */
     void uninstallConfiguration(Artifact configurationId) throws IOException, NoSuchConfigException;
 
+    /**
+     * Gets the common ArtifactResolver in case the caller wants to use this
+     * directly.  It is configured for all the repositories known to this
+     * configuration manager, etc.
+     */
     ArtifactResolver getArtifactResolver();
 }

Modified: geronimo/branches/1.1/modules/kernel/src/java/org/apache/geronimo/kernel/repository/ArtifactResolver.java
URL: http://svn.apache.org/viewcvs/geronimo/branches/1.1/modules/kernel/src/java/org/apache/geronimo/kernel/repository/ArtifactResolver.java?rev=396024&r1=396023&r2=396024&view=diff
==============================================================================
--- geronimo/branches/1.1/modules/kernel/src/java/org/apache/geronimo/kernel/repository/ArtifactResolver.java (original)
+++ geronimo/branches/1.1/modules/kernel/src/java/org/apache/geronimo/kernel/repository/ArtifactResolver.java Fri Apr 21 15:26:18 2006
@@ -88,6 +88,8 @@
      * Used to search for existing artifacts in the server that match the supplied
      * artifact (which may be partially-populated).
      *
+     * TODO: The artifacts should be sorted ascending by type then group then artifact then version
+     *
      * @return The matching artifacts, which may be 0, 1, or many
      */
     Artifact[] queryArtifacts(Artifact artifact);