<div dir="ltr"><div><div><div><div>You are absolutely right!<br><br></div>Don't know how I slipped that one... and how come testsuite did not catch this.<br><br></div>Same as for web also goes for infinispan and jgroups.<br>
<br></div>This 3 subsystems only have msc dependencies to threads subsystem...<br><br></div><div>I think we can revisit this for AS8, aka make it the same way as jca & ejb3 use this.<br></div><div><br>--<br></div>tomaz<br>
</div><div class="gmail_extra"><br><br><div class="gmail_quote">On Fri, Feb 1, 2013 at 5:04 PM, Andrig Miller <span dir="ltr"><<a href="mailto:anmiller@redhat.com" target="_blank">anmiller@redhat.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div><div style="font-size:12pt;font-family:arial,helvetica,sans-serif">Actually the thread subsystem is needed to be able to create a sane thread pool for JBoss Web. You cannot rely on its internal thread pool implementation to get a properly configured thread pool. You must use its "executor" attribute, and define a thread pool in the thread subsystem.<br>
<br>This<span style="font-weight:bold"> CANNOT BE REMOVED</span>, its absolutely critical.<br><br>Andy<br><br><hr><blockquote style="padding-left:5px;font-size:12pt;font-style:normal;margin-left:5px;font-family:Helvetica,Arial,sans-serif;text-decoration:none;font-weight:normal;border-left:2px solid rgb(16,16,255)">
<b>From: </b>"Tomaž Cerar" <<a href="mailto:tomaz.cerar@gmail.com" target="_blank">tomaz.cerar@gmail.com</a>><br><b>To: </b>"JBoss AS7 Development" <<a href="mailto:jboss-as7-dev@lists.jboss.org" target="_blank">jboss-as7-dev@lists.jboss.org</a>><br>
<b>Sent: </b>Friday, February 1, 2013 8:42:09 AM<br><b>Subject: </b>[jboss-as7-dev] Removing thread subsystem<div><div class="h5"><br><br><div dir="ltr"><div><div><div><div><div><div><div>Hi,<br><br></div>Threads subsystem has not been directly used since 7.1.0.CR1.<br>
</div>see <a href="https://issues.jboss.org/browse/AS7-390" target="_blank">https://issues.jboss.org/browse/AS7-390</a> for more details.<br>
<br><div>The only usage we have is that EJB3 & JCA subsystems use some
common code from threads subsystem (parser & few RD) but beyond that
subsystem is unused aka we could remove ThreadExtension and everything
would still work.<br></div><br>And ever since there was no need for thread subsystem enabled/deployed in configuration but we still had it in all our default shipped configs.<br></div><div><br>I would like to remove it from our default shipped configurations as it does not effect any functionality.<br>
<br></div>Also I have introduced option to "deprecate" extension, so it now logs warning when such extension is registered (aka is found in configuration)<br><br></div>As goes for compatibility issues in mixed-domain this change does not effect 7.1.0+/EAP6 as they can properly operate without this subsystem.<br>
<br></div><div>It looks like that we removed usage of Threads subsystem in 7.1.0.CR1 but forgot to remove it from shipped configurations.<br></div><div><br></div>my current effort at this can be found at <a href="https://github.com/ctomc/jboss-as/compare/threads" target="_blank">https://github.com/ctomc/jboss-as/compare/threads</a><br>
</div>(there is lots of removed redundant & old style tests)<br><br><br>--<br></div>tomaz<br></div>
<br></div></div>_______________________________________________<br>jboss-as7-dev mailing list<br><a href="mailto:jboss-as7-dev@lists.jboss.org" target="_blank">jboss-as7-dev@lists.jboss.org</a><br><a href="https://lists.jboss.org/mailman/listinfo/jboss-as7-dev" target="_blank">https://lists.jboss.org/mailman/listinfo/jboss-as7-dev</a><br>
</blockquote><br></div></div></blockquote></div><br></div>