diff -u -r1.1 -r1.2
--- ConditionsRecordCollection.java	24 Sep 2013 02:40:21 -0000	1.1
+++ ConditionsRecordCollection.java	24 Sep 2013 02:51:35 -0000	1.2
@@ -26,7 +26,8 @@

     }
         
     /**

-     * Find a ConditionsRecord by its name or key.  This is the 'name' field from the conditions database table. 

+     * Find a ConditionsRecord by its name or key in this collection.  
+     * This is the 'name' field from the conditions database table. 

      * @param name The name of the conditions set, e.g. 'svt_calibrations' etc.
      * @return The collection of ConditionsRecords, which can be empty if none were found.
      */

diff -u -r1.7 -r1.8
--- DatabaseConditionsReader.java	24 Sep 2013 02:40:21 -0000	1.7
+++ DatabaseConditionsReader.java	24 Sep 2013 02:51:35 -0000	1.8
@@ -3,9 +3,7 @@

 import java.io.IOException;
 import java.io.InputStream;
 import java.sql.Connection;

-import java.sql.ResultSet;

 import java.sql.SQLException;

-import java.sql.Statement;

 import java.util.logging.ConsoleHandler;
 import java.util.logging.Formatter;
 import java.util.logging.Level;

@@ -31,10 +29,10 @@

  * 
  * Currently, this class should "know" directly about all the converters that are needed for loading
  * conditions data via the <code>registerConditionsConverters</code> method.  The SVT is the only example 

- * currently, but more will be added as they are created.

+ * currently of an actual HPS sub-system, but more will be added as they are created.

  * 
  * @author jeremym

- * @version $Id: DatabaseConditionsReader.java,v 1.7 2013/09/24 02:40:21 jeremy Exp $ 

+ * @version $Id: DatabaseConditionsReader.java,v 1.8 2013/09/24 02:51:35 jeremy Exp $ 

  */
 public class DatabaseConditionsReader extends ConditionsReader {
         

@@ -105,9 +103,7 @@

      * @param run The run number.
      */
     public boolean update(ConditionsManager manager, String detectorName, int run) throws IOException {

-        
-        System.out.println("update");
-        

+                

         logger.info("update");
         logger.info("  detectorName: " + detectorName);
         logger.info("  run: " + run);

@@ -123,11 +119,11 @@

         
         // Check if conditions for this run are already loaded.
         if (run <= maxRun && run >= minRun) {

-            logger.info("conditions already cached");

+            logger.info("Conditions already cached for this run.");

             return false;
         }
         

-        // Register the converters on the manager.

+        // Register the converters on the manager.  

         registerConditionsConverters(manager);
                 
         // Open a connection to the database.

@@ -152,22 +148,23 @@

     }
 
     /**

-     * Close the base reader of the detector description.

+     * Close the base reader.

      */
     public void close() throws IOException {
         reader.close();
     }
 
     /**

-     * Read properties from cache, which are table, field and id.
-     * @return An InputStream with the cached properties for <code>name</code>.

+     * Implementation of ConditionReader API method.
+     * @return An InputStream with the conditions for <code>type</code>.

      */
     public InputStream open(String name, String type) throws IOException {
         return reader.open(name, type);
     }
      
     /**

-     * This will cache the run numbers so that the actual data can be fetched later.

+     * This will cache the run number range.  More could happen here but conditions
+     * data is generally loaded lazily (as needed).

      * @param run The run number.
      * @throws SQLException
      * @throws IOException