<!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: separating deployers from other services?
</h3>
<span style="margin-bottom: 10px;">
created by <a href="http://community.jboss.org/people/bstansberry%40jboss.com">Brian Stansberry</a> in <i>JBoss AS7 Development</i> - <a href="http://community.jboss.org/message/564345#564345">View the full discussion</a>
</span>
<hr style="margin: 20px 0; border: none; background-color: #dadada; height: 1px;">
<div class="jive-rendered-content"><blockquote class="jive-quote"><p>Emanuel Muckenhuber wrote:</p><p style="min-height: 8pt; height: 8pt; padding: 0px;"> </p>Where the important piece is that deployers don't inject services directly rather in the services they deploy.</blockquote><p style="min-height: 8pt; height: 8pt; padding: 0px;"> </p><p>I think what you are saying here is, say we have a mythical WebServer service that's the entire runtime webserver. A mythical WarDeployer service should not have that injected into it and programatically inject it into a mythical WarDeploymentService it's working on. Rather WarDeploymentService should expose an Injector<WebServer> and  WarDeploymentService should do something like this:</p><p style="min-height: 8pt; height: 8pt; padding: 0px;"> </p><p>BatchServiceBuilder<WarDeploymentService> builder = .... came from somewhere</p><p>WarDeploymentService warService = .... came from somewhere</p><p>builder.addDependency(WebServer.SERVICE_NAME, WebServer.class, warService.getWebServerInjector());</p><p style="min-height: 8pt; height: 8pt; padding: 0px;"> </p><p>Agreed on this.</p><p style="min-height: 8pt; height: 8pt; padding: 0px;"> </p><p>Your 300 ms improvement by doing deployments and non-deployer services in parallel is good evidence that your basic idea is correct. The trick is we need to sort out how to do it. Your patch impl works with the current way things are done, buy David is looking at ditching batches as a result of MSC-21. That needs to be discussed in full before we do it and the implications fully understood. Seems that discussion needs to happen right now.</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/564345#564345">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>