I'll take a look at this tonight. Repetitive logging should be possible to eliminate.
I'm not at all convinced that stopping deployment of anything except the mbean that won't finish is desirable, practical, or possible. Even that mbean I'm not so sure of. If you are deploying many components at once I think rolling back anything due to one error is undesirable. david jencks On 2002.08.30 16:32:56 -0400 Dain Sundstrom wrote: > My startup code throws DeploymentExceptions if an error is detected in > the configuration, and this used to cause deployment to stop. I doesn't > anymore. > > In org.jboss.system.ServiceController.start(ObjectName) we have the > following code on line 366 (HEAD): > > // Call start on the service Proxy > try > { > ctx.proxy.start(); > } > catch (Throwable e) > { > ctx.checkTransition(ServiceContext.NOTSTARTABLE); > ctx.problem = e; > log.warn("Problem starting service " + serviceName, e); > return; > } > > This simply writes an unnecessary exception trace and allows the > deployment to continue. > > Is there a reason for this code? If not I will be more then happy > remove the try/ catch and allow the code to exception out. > > > BTW: If you want to reproduce this error modify the > src/resources/cmp2/commerce/META-INF/jbosscmp-jdbc.xml and add some junk > to one of the cmp-field names (like change userName to userNameXXXXX). > Then run the following command: > > ./build.sh -Dtest=org.jboss.test.cmp2.commerce.CommerceTest one-test > > Watch the 30 pages of exceptions flow by and the app still deploys. > > -- > xxxxxxxxxxxxxxxxxxxxxxxx > Dain Sundstrom > Chief Architect JBossCMP > JBoss Group, LLC > xxxxxxxxxxxxxxxxxxxxxxxx > > > > ------------------------------------------------------- > This sf.net email is sponsored by: OSDN - Tired of that same old > cell phone? Get a new here for FREE! > https://www.inphonic.com/r.asp?r=sourceforge1&refcode1=vs3390 > _______________________________________________ > Jboss-development mailing list > [EMAIL PROTECTED] > https://lists.sourceforge.net/lists/listinfo/jboss-development > > ------------------------------------------------------- This sf.net email is sponsored by: OSDN - Tired of that same old cell phone? Get a new here for FREE! https://www.inphonic.com/r.asp?r=sourceforge1&refcode1=vs3390 _______________________________________________ Jboss-development mailing list [EMAIL PROTECTED] https://lists.sourceforge.net/lists/listinfo/jboss-development