You are viewing a plain text version of this content. The canonical link for it is here.
Posted to ojb-dev@db.apache.org by ar...@apache.org on 2003/05/06 16:09:27 UTC

cvs commit: db-ojb/src/java/org/apache/ojb/broker/accesslayer ConnectionFactoryAbstractImpl.java

arminw      2003/05/06 07:09:27

  Modified:    src/java/org/apache/ojb/broker/accesslayer
                        ConnectionFactoryAbstractImpl.java
  Log:
  add more logging info
  
  Revision  Changes    Path
  1.5       +6 -5      db-ojb/src/java/org/apache/ojb/broker/accesslayer/ConnectionFactoryAbstractImpl.java
  
  Index: ConnectionFactoryAbstractImpl.java
  ===================================================================
  RCS file: /home/cvs/db-ojb/src/java/org/apache/ojb/broker/accesslayer/ConnectionFactoryAbstractImpl.java,v
  retrieving revision 1.4
  retrieving revision 1.5
  diff -u -r1.4 -r1.5
  --- ConnectionFactoryAbstractImpl.java	5 May 2003 23:46:42 -0000	1.4
  +++ ConnectionFactoryAbstractImpl.java	6 May 2003 14:09:27 -0000	1.5
  @@ -104,10 +104,11 @@
               {
                   if (conn == null || conn.isClosed())
                   {
  -                    log.error("Connection for JdbcConnectionDiscriptor ("
  -                            + jcd.getDatasourceName() != null ? "datasource: " + jcd.getDatasourceName() :
  -                            "db-url: " + getDbURL(jcd) + ", user: " + jcd.getUserName() +
  -                            ") was not sane, either closed or null");
  +                    log.error("Connection for JdbcConnectionDiscriptor (" +
  +                            (jcd.getDatasourceName() != null ? "datasource: " + jcd.getDatasourceName() :
  +                            "db-url: " + getDbURL(jcd) + ", user: " + jcd.getUserName()) +
  +                            ") was not valid, either *closed* or *null*");
  +                    throw new LookupException("Could not lookup valid connection from pool/DB, connection was "+conn);
                   }
               }
               catch (SQLException e)