[jBoss-Dev] CVS update:jboss/src/main/org/jboss/ejb/plugins/jaws/jdbc JDBCCommand.javaJDBCFindByCommand.java JDBCQueryCommand

[jBoss CVS Development]

  User: fleury  
  Date: 00/08/17 20:21:02

  Modified:    src/main/org/jboss/ejb/plugins/jaws/jdbc JDBCCommand.java
                        JDBCFindByCommand.java JDBCQueryCommand.java
  Log:
  Logger messages and clean up for binary
  
  Revision  Changes    Path
  1.2       +7 -5      jboss/src/main/org/jboss/ejb/plugins/jaws/jdbc/JDBCCommand.java
  
  Index: JDBCCommand.java
  ===================================================================
  RCS file: 
/products/cvs/ejboss/jboss/src/main/org/jboss/ejb/plugins/jaws/jdbc/JDBCCommand.java,v
  retrieving revision 1.1
  retrieving revision 1.2
  diff -u -r1.1 -r1.2
  --- JDBCCommand.java  2000/08/06 02:03:49     1.1
  +++ JDBCCommand.java  2000/08/18 03:21:01     1.2
  @@ -32,7 +32,9 @@
   import org.jboss.ejb.plugins.jaws.deployment.JawsEntity;
   import org.jboss.ejb.plugins.jaws.deployment.JawsCMPField;
   import org.jboss.logging.Log;
  +import org.jboss.logging.Logger;
   
  +
   /**
    * Abstract superclass for all JAWS Commands that use JDBC directly.
    * Provides a Template Method for jdbcExecute(), default implementations
  @@ -41,7 +43,7 @@
    *
    * @see <related>
    * @author <a href="mailto:[EMAIL PROTECTED]">Justin Forder</a>
  - * @version $Revision: 1.1 $
  + * @version $Revision: 1.2 $
    */
   public abstract class JDBCCommand
   {
  @@ -97,7 +99,7 @@
                  stmt.close();
               } catch (SQLException e)
               {
  -               e.printStackTrace();
  +               Logger.exception(e);
               }
            }
            if (con != null)
  @@ -107,7 +109,7 @@
                  con.close();
               } catch (SQLException e)
               {
  -               e.printStackTrace();
  +               Logger.exception(e);
               }
            }
         }
  @@ -233,7 +235,7 @@
         } catch (Exception e)
         {
            // JF: Dubious - better to throw a meaningful exception
  -         e.printStackTrace();
  +         Logger.exception(e);
            return Types.OTHER;
         }
      }
  @@ -368,7 +370,7 @@
               jdbcTypeNames.put(fieldValue, fieldName);
            } catch (IllegalAccessException e) {
               // Should never happen
  -            e.printStackTrace();
  +            Logger.exception(e);
            }
         }
      }
  
  
  
  1.2       +4 -2      
jboss/src/main/org/jboss/ejb/plugins/jaws/jdbc/JDBCFindByCommand.java
  
  Index: JDBCFindByCommand.java
  ===================================================================
  RCS file: 
/products/cvs/ejboss/jboss/src/main/org/jboss/ejb/plugins/jaws/jdbc/JDBCFindByCommand.java,v
  retrieving revision 1.1
  retrieving revision 1.2
  diff -u -r1.1 -r1.2
  --- JDBCFindByCommand.java    2000/08/06 02:03:50     1.1
  +++ JDBCFindByCommand.java    2000/08/18 03:21:01     1.2
  @@ -16,7 +16,9 @@
   import org.jboss.ejb.plugins.jaws.deployment.Finder;
   import org.jboss.ejb.plugins.jaws.CMPFieldInfo;
   import org.jboss.ejb.plugins.jaws.deployment.JawsCMPField;
  +import org.jboss.logging.Logger;
   
  +
   /**
    * JAWSPersistenceManager JDBCFindByCommand
    *
  @@ -25,7 +27,7 @@
    * @author <a href="mailto:[EMAIL PROTECTED]">Marc Fleury</a>
    * @author <a href="mailto:[EMAIL PROTECTED]">Joe Shevland</a>
    * @author <a href="mailto:[EMAIL PROTECTED]">Justin Forder</a>
  - * @version $Revision: 1.1 $
  + * @version $Revision: 1.2 $
    */
   public class JDBCFindByCommand extends JDBCFinderCommand
   {
  @@ -45,7 +47,7 @@
         super(factory, finderMethod.getName());
         
         String cmpFieldName = finderMethod.getName().substring(6).toLowerCase();
  -      System.out.println("Finder:"+cmpFieldName);
  +      Logger.log("Finder:"+cmpFieldName);
         
         // Find the meta-info for the field we want to find by
         
  
  
  
  1.2       +4 -2      
jboss/src/main/org/jboss/ejb/plugins/jaws/jdbc/JDBCQueryCommand.java
  
  Index: JDBCQueryCommand.java
  ===================================================================
  RCS file: 
/products/cvs/ejboss/jboss/src/main/org/jboss/ejb/plugins/jaws/jdbc/JDBCQueryCommand.java,v
  retrieving revision 1.1
  retrieving revision 1.2
  diff -u -r1.1 -r1.2
  --- JDBCQueryCommand.java     2000/08/06 02:03:51     1.1
  +++ JDBCQueryCommand.java     2000/08/18 03:21:01     1.2
  @@ -10,14 +10,16 @@
   import java.sql.PreparedStatement;
   import java.sql.ResultSet;
   import java.sql.SQLException;
  +import org.jboss.logging.Logger;
   
  +
   /**
    * Abstract superclass for all JAWS Commands that issue JDBC queries
    * directly.
    * Provides a Template Method implementation for
    * <code>executeStatementAndHandleResult</code>.
    * @author <a href="mailto:[EMAIL PROTECTED]">Justin Forder</a>
  - * @version $Revision: 1.1 $
  + * @version $Revision: 1.2 $
    */
   public abstract class JDBCQueryCommand extends JDBCCommand
   {
  @@ -54,7 +56,7 @@
                  rs.close();
               } catch (SQLException e)
               {
  -               e.printStackTrace();
  +               Logger.exception(e);
               }
            }
         }