testdriven / psi-probe

Advanced manager and monitor for Apache Tomcat, forked from Lambda Probe
8 stars 1 forks source link

DataSources don't show BoneCPDatasource (BoneCP 0.8.0.RELEASE) #405

Open padcom opened 9 years ago

padcom commented 9 years ago

From w...@albamoda.de on April 24, 2014 13:58:07

Hi,

The BoneCPDatasourceAccessor doesn't work with the newest BoneCP Release (0.8.0-RELEASE). There is a new Wrapper class (FinalWrapper) in BoneCPDataSource, so the cast to BoneCP is broken:

final Field poolField = BoneCPDataSource.class.getDeclaredField("pool"); poolField.setAccessible(true); final BoneCP pool = (BoneCP) poolField.get(source);

The Exception is: java.lang.ClassCastException: com.jolbox.bonecp.BoneCPDataSource$FinalWrapper cannot be cast to com.jolbox.bonecp.BoneCP

There is a new getter in BoneCPDataSource to get the Pool directly, so

final BoneCP pool = source.getPool();

should do the work know.

I'm using probe version 2.3.3 on Tomcat 7.0.42 with jdk 1.7.0_25

Kind regards Jan-Hendrik

Original issue: http://code.google.com/p/psi-probe/issues/detail?id=405

padcom commented 9 years ago

From MALfunct...@gmail.com on May 13, 2014 06:29:48

BoneCP support came from the community ( issue 334 ). A patch would be much appreciated.

Status: NeedInfo
Labels: HelpWanted

padcom commented 9 years ago

From MALfunct...@gmail.com on September 08, 2014 08:02:33

I'm going to take a stab at this.

Status: Started
Labels: -Milestone-TBD -HelpWanted Milestone-2.4.0

padcom commented 9 years ago

From MALfunct...@gmail.com on September 08, 2014 08:07:05

This issue was updated by revision r761 .

Fixing exception thrown when using BoneCP 0.8.0.

Status: FixedInSource

padcom commented 9 years ago

From MALfunct...@gmail.com on September 08, 2014 08:07:58

Can someone verify that this worked?

Owner: MALfunct...@gmail.com
Labels: Verified-No