RC1: Resource Adaptor configuration of non-String types is b

[mprudhom]

I posted a bug report with at patch 3 months ago at:

http://sourceforge.net/tracker/index.php?func=detail&aid=518896&group_id=22866&atid=376685

The problem is that if any of the config-property-type values are something other than a String, then the configuration will fail. E.g, if I have:

<config-property> <config-property-name>MinPool</config-property-name>
<config-property-type>java.lang.Integer</config-property-type>
<config-property-value>1</config-property-value>
</config-property>

It will result in the log message:

2002-05-09 15:11:29,512 WARN [org.jboss.resource.connectionmanager.RARDeployment] Unable to find a PropertyEditor for class 'class java.lang.Integer' of property 'MinPool' - skipping property


It is pretty critical to us that this fix gets into the final 3.0.0 release: is there anything I can do to expidite the application of my very simple patch?

[davidjencks]

I'll take a look.

[dezzio]

Any news on this?

[davidjencks]

Hmm, this was working for me for a while but I don't seem to have a test case. If this is still broken please file a bug including your version info.

[davidjencks]

I've just added some test code to 3.1 with Integer, Boolean, Double, and URL properties, all of which appear to work fine. If you are having trouble please be very specific.

[neil]

I get the same problem as the others in 3.0.0. What files do you need to see to prove this is happening? I can give you my full RA code if necessary.

- Neil

[davidjencks]

Could you see if your adapter deploys ok in jboss 3.1? If not please compare with the test adapter in org.jboss.test.jca.adapter, which is what the test case is based on.

WARNING: I just changed some configuration in 3.1, the JndiName attribute now goes with the ConnectionManager mbean, not the RARDeployment (usually labeled service=LocalTxDS or some such)

It is possible that the code that makes the property editors available only got into jboss 3.0.1, so you might try that or either cvs version also.