That was a conscious decision. If we are going to get to 24x7
available kernels it means never restarting the server. Bad apps
should be fixable without taking down the server. The warning
log message is necessary if an admin is to be informed that
a deployment has failed. Otherwise your letting your customers
perform you administration by telling your app is broke.
xxxxxxxxxxxxxxxxxxxxxxxx
Scott Stark
Chief Technology Officer
JBoss Group, LLC
xxxxxxxxxxxxxxxxxxxxxxxx
----- Original Message -----
From: "Dain Sundstrom" <[EMAIL PROTECTED]>
To: "JBoss-dev" <[EMAIL PROTECTED]>
Sent: Friday, August 30, 2002 1:32 PM
Subject: [JBoss-dev] DeploymentException does NOT stop deployment
> 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
