Avi Cherry wrote:
> I recommend that you fix the obviously broken code right away and
> commit the changes. I think we should decide what to do after that.
I fixed the bug and added some simple documentation (I also converted
tabs to 8 spaces on this one file so I could make read the code and spot
any problems... so the CVS diff looks more sweeping than it really is).
I still want to change the approach some as I had mentioned earlier
(faster startup, more aspects configurable). Now that we're defining
the connection from a properties file, many more settings can be
configurable (like # of attempts to initially connect to database before
failing, time to sleep while blocking while waiting for an available
connection, SQL statement to use to test whether a connection is still
valid, and other things).
I'll hold off on any of these changes as I'd like to see the code
working now more reliably for Avi before I start changing anything
else. Hopefully with the race condition fixed, you won't get the server
hanging anymore. Let me know if you have problems.
Serge Knystautas
Loki Technologies
http://www.lokitech.com/
------------------------------------------------------------
To subscribe: [EMAIL PROTECTED]
To unsubscribe: [EMAIL PROTECTED]
Problems?: [EMAIL PROTECTED]
