Hi,
I found this problem back in early march and submitted a patch by email. 
Apparently in the move to sourceforge it got lost.  I've resubmitted it to
the sourceforge patch page. Patch # is 424399. Christian, you might add the
bug # to the patch if you submit a bug.  This patch is against 2.3 cvs but
the same modifications worked with 2.1.

My theory is that if you stop a XADatasource, you want a new copy when you
restart it.  At least the pools don't seem to be amenable to
reconfiguration.  So my fix frees the old source when the mbean is stopped,
and a new one is created when the mbean is restarted. Also, if the
connection can't be established when the pool is initialized, it's freed so
you can change the configuration and try again without restarting the
server. 

Thanks
david jencks

On 2001.05.15 15:39:16 -0400 Toby Allsopp wrote:
> On Tue, May 15, 2001 at 01:35:22PM +0200, Christian Biasuzzi wrote:
> > Hi,
> >  I'm playing with the latest jboss CVS snapshot
> >     and I found yesterday that I'm not able to restart a XADataSource
> > MBean (the DefaultDS, for example) after having stopped it
> > 
> >  here follows my log:
> > 
> > [DefaultDS] java.lang.NullPointerException
> > [DefaultDS]     at
> > org.jboss.pool.jdbc.xa.XAPoolDataSource.setPoolName(XAPoolDataSource.java:96)
> > 
> > [DefaultDS]     at
> > org.jboss.jdbc.XADataSourceLoader.startService(XADataSourceLoader.java:267)
> > 
> > [DefaultDS]     at
> > org.jboss.util.ServiceMBeanSupport.start(ServiceMBeanSupport.java:101)
> > [DefaultDS]     at java.lang.reflect.Method.invoke(Native Method)
> > [DefaultDS]     at
> > com.sun.management.jmx.MBeanServerImpl.invoke(MBeanServerImpl.java:1628)
> > 
> >    .....................
> > 
> >   If  I put a
> > 
> >      source = null;
> > 
> >  at the end of stopService() in XADataSourceLoader class
> >   (actually forcing the creation of a new XADataSource) everything runs
> > fine, but I'm not sure this is the
> >   right thing to do....
> 
> Please log this as a bug on SourceForge.  I'll have a look at it at some
> point
> if no one else does.
> 
> Thanks,
> 
> Toby.
> 
> _______________________________________________
> JBoss-user mailing list
> [EMAIL PROTECTED]
> http://lists.sourceforge.net/lists/listinfo/jboss-user
> 


_______________________________________________
JBoss-user mailing list
[EMAIL PROTECTED]
http://lists.sourceforge.net/lists/listinfo/jboss-user

Reply via email to