Re: [Proxool-developer] New listener stuff

[Bill H. <bi...@lo...>]

Chr,

On Fri, 2003-02-07 at 02:02, Christian Nedregaard wrote:

> The new classes consists of an abstract generic listener container
> that supports concurency with a read/write lock pluss concrete
> implementations for each listener type. 

Great stuff chr.

> Bill - There is one test failure at the moment:
> 1) testConfigurator(org.logicalcobwebs.proxool.configuration.ConfiguratorTest junit.framework.AssertionFailedError: adapter.getConnectionPoolDefinition() is null
> 
> ...I hope this is not my fault =8-0

No. The reason it happens is that that tests uses a pool that has been
registered and removed. When I changed the alias used in that test from
"template" to "configurator" it worked fine. 

We *should* be able to register, remove and then re-register a pool so I
have written a new unit test for exactly that. This showed that the
Properties are not used when you re-register (so the connection to
hsqldb fails because the user is not set). It is now fixed.

> I've also added a new "util" package for generic utils that might be
> required outside of the main proxool package. I've moved FastArrayList
> there.

Good.

Bill