search

DanElbert / vt-middleware

Automatically exported from code.google.com/p/vt-middleware
0 stars 0 forks source link

ldaptive: pool scheduled tasks may die if LDAP unavailable #158

Closed GoogleCodeExporter closed 8 years ago

GoogleCodeExporter commented 8 years ago 
If the pool is configured to validate periodically and that validation removes 
all connections from the pool, the subsequent call to maintain a minimum number 
of connections in the pool will throw IllegalStateException.
This kills the validation task until that pool is initialized again.

Original issue reported on code.google.com by dfis...@gmail.com on 22 May 2013 at 10:29

GoogleCodeExporter commented 8 years ago 
Made the following changes:
  * refactored #initializePool() to #grow(int)
  * #initializePool() no longer throws IllegalStateException, that condition is checked and thrown from #initialize()
  * surround scheduled executors with try-catch
  * updated #initialize to with an additional sanity check for validator configuration
  * improved logging by including the pool in more logging statements

Fixed in r2727.

Original comment by dfis...@gmail.com on 24 May 2013 at 8:26