Martin, Thanks for the note about the recent change. I'll try to see exactly which version it changed in.
However, I'm still concerned about the mechanism for waiting. It is 30 'wait cycles' where each wait cycle would complete from one of the following conditions: 1) the connection becomes available 2) the 1000ms wait call completes and the thread resumes 3) the notifyAll method is called causing the wait to return immediately (not just after 1000ms). Under high contention for those connections I believe two things can happen: - starvation of access to a connection because the waiting mechanism is unfair - waiting potentially way less than 30 seconds because the waiting mechanism isn't really much about time at all (it is about wait/ notifyAll cycles in high contention). I might just be reading the code wrong or misunderstand the wait/ notifyAll behavior...but I'm quite sure I'm seeing these 'Login timeout' exceptions at times way before 30 seconds hit. Thanks Joe On Aug 5, 5:09 am, Wildam Martin <[email protected]> wrote: > On Thu, Aug 5, 2010 at 06:51, Joe <[email protected]> wrote: > > Couple of thoughts: > > 1) The JDBCConnectionPool setLoginTimeout method says the default > > timeout is 5 minutes. However, the default is actually set to '30' in > > the code. That appears that is supposed to be '30' seconds. > > The default timeout has been reduced to 30 seconds recently. It seems > that the javadoc has not been updated yet. It has been reduced because > for a lot of people the 5 minutes timeout was too long if e.g. a > server is temporarily not available because only after 5 minutes a > message could be presented to the user that the connection could not > be established. > -- > Martin Wildamhttp://www.google.com/profiles/mwildam -- You received this message because you are subscribed to the Google Groups "H2 Database" group. To post to this group, send email to [email protected]. To unsubscribe from this group, send email to [email protected]. For more options, visit this group at http://groups.google.com/group/h2-database?hl=en.
