<!DOCTYPE HTML PUBLIC "-//W3C//DTD HTML 4.0 Transitional//EN">
<html>
<body link="#355491" alink="#4262a1" vlink="#355491" style="background: #e2e2e2; margin: 0; padding: 20px;">
<div>
<table cellpadding="0" bgcolor="#FFFFFF" border="0" cellspacing="0" style="border: 1px solid #dadada; margin-bottom: 30px; width: 100%; -moz-border-radius: 6px; -webkit-border-radius: 6px;">
<tbody>
<tr>
<td>
<table border="0" cellpadding="0" cellspacing="0" bgcolor="#FFFFFF" style="border: solid 2px #ccc; background: #dadada; width: 100%; -moz-border-radius: 6px; -webkit-border-radius: 6px;">
<tbody>
<tr>
<td bgcolor="#000000" valign="middle" height="58px" style="border-bottom: 1px solid #ccc; padding: 20px; -moz-border-radius-topleft: 3px; -moz-border-radius-topright: 3px; -webkit-border-top-right-radius: 5px; -webkit-border-top-left-radius: 5px;">
<h1 style="color: #333333; font: bold 22px Arial, Helvetica, sans-serif; margin: 0; display: block !important;">
<!-- To have a header image/logo replace the name below with your img tag -->
<!-- Email clients will render the images when the message is read so any image -->
<!-- must be made available on a public server, so that all recipients can load the image. -->
<a href="http://community.jboss.org/index.jspa" style="text-decoration: none; color: #E1E1E1">JBoss Community</a></h1>
</td>
</tr>
<tr>
<td bgcolor="#FFFFFF" style="font: normal 12px Arial, Helvetica, sans-serif; color:#333333; padding: 20px; -moz-border-radius-bottomleft: 4px; -moz-border-radius-bottomright: 4px; -webkit-border-bottom-right-radius: 5px; -webkit-border-bottom-left-radius: 5px;"><h3 style="margin: 10px 0 5px; font-size: 17px; font-weight: normal;">
Re: Asynchronous results from executing a deployment plan
</h3>
<span style="margin-bottom: 10px;">
created by <a href="http://community.jboss.org/people/david.lloyd%40jboss.com">David Lloyd</a> in <i>JBoss AS7 Development</i> - <a href="http://community.jboss.org/message/560743#560743">View the full discussion</a>
</span>
<hr style="margin: 20px 0; border: none; background-color: #dadada; height: 1px;">
<div class="jive-rendered-content"><p>I don't like the await() methods.  Talking from MSC experience, using a caller-blocking methodology doesn't mix well with the callback-driven methodology used by MSC.  In particular, if you use an await()-like method from within an MSC task (listener, service start/stop, etc) which depends on a change in another service (and believe me, people will do this), you may be introducing a deadlock since that service's completion may ultimately depend upon you.</p><p style="min-height: 8pt; height: 8pt; padding: 0px;"> </p><p>That said - looks like you have no provision for supporting on-demand services.  This is actually a tricky problem because a service's mode can change irrespective of the current controller state (i.e. the fact that a listener is running doesn't "preserve" the current controller mode).  Thus a service's mode can change on you, so you can't just say "if this service is automatic/immediate, add it to the set of incomplete dependents" because changing the mode to on-demand can "complete" it.  Also, an AUTOMATIC service which doesn't start because of an ON_DEMAND dependency could also be considered "complete" for the purposes of deployment.  However not all AUTOMATIC services can be considered complete.</p><p style="min-height: 8pt; height: 8pt; padding: 0px;"> </p><p>So the question is, what does it mean for a deployment to be "done"?  Saying that all services in the deployment have fully started won't work due to on-demand and other services which are not expected to start immediately, and possibly changing modes etc.  So you really need to track a specific subset of services which represent the meat of the deployment, which can vary based on deployment type and probably other factors as well.  For some deployments, it may not even be possible to wait for <em>any</em> services to start.  This means that the listener would be applied to a select set of services only.</p><p style="min-height: 8pt; height: 8pt; padding: 0px;"> </p><p>As for the race condition, that's not too hard to solve since you're using locks & conditions: just add a flag "started" which you set to "true" (with a signalAll() [btw, you should use signal*, not notify* with Condition]) which is evaluated as part of the condition for readiness.  The flag would be set once the batch is installed (then you'd know all the listeners were added).</p></div>
<div style="background-color: #f4f4f4; padding: 10px; margin-top: 20px;">
<p style="margin: 0;">Reply to this message by <a href="http://community.jboss.org/message/560743#560743">going to Community</a></p>
<p style="margin: 0;">Start a new discussion in JBoss AS7 Development at <a href="http://community.jboss.org/choose-container!input.jspa?contentType=1&containerType=14&container=2225">Community</a></p>
</div></td>
</tr>
</tbody>
</table>
</td>
</tr>
</tbody>
</table>
</div>
</body>
</html>