I reviewed and tested your patch. On my installation, it works fine with MySQL,
Sybase ASE and WebLogic pool driver running on top of the other two.
It's good that the .properties file values are used properly now.
You have a +1 from me.
While testing this stuff on WebLogic 5.1 I made a workaround for an obscure
bug in it's classloader. When you try to use SybDriver class for the first time,
it throws ClassCircularityError at you. All the following attempts work like no
problem. I made catch - retry block to handle that situation. It works, but
it seems that there are more issues with this classloader: upon the first hit
to Turbine, I get ClassCircularityError on org.apache.ecs.StringElement.
Supporting closed software is no fun.
Rafal
--- DB.java.orig Sun Jun 4 00:56:30 2000
+++ DB.java Sun Jun 4 01:36:49 2000
@@ -86,7 +86,16 @@
/** returns a connection from the DriverManager */
public Connection getConnection() throws SQLException
{
- return DriverManager.getConnection( DB_CONNECTION, DB_USER, DB_PASS );
+ // workaround for buggy WebLogic 5.1 classloader - ignore the exception
+ // upon first invocation
+ try
+ {
+ return DriverManager.getConnection( DB_CONNECTION, DB_USER, DB_PASS );
+ }
+ catch( ClassCircularityError e )
+ {
+ return DriverManager.getConnection( DB_CONNECTION, DB_USER, DB_PASS );
+ }
}
/** basic initialization
------------------------------------------------------------
To subscribe: [EMAIL PROTECTED]
To unsubscribe: [EMAIL PROTECTED]
Problems?: [EMAIL PROTECTED]
