<html><head><style type='text/css'>p { margin: 0; }</style></head><body><div style='font-family: arial,helvetica,sans-serif; font-size: 12pt; color: #000000'>No, it wouldn't be a bad thing to have to add it, although it wouldn't be obvious what schema version you should put in place, without the empty definition in place already.<br><br>Andy<br><br><hr id="zwchr"><blockquote style="border-left: 2px solid rgb(16, 16, 255); margin-left: 5px; padding-left: 5px; color: rgb(0, 0, 0); font-weight: normal; font-style: normal; text-decoration: none; font-family: Helvetica,Arial,sans-serif; font-size: 12pt;"><b>From: </b>"Tomaž Cerar" <tomaz.cerar@gmail.com><br><b>To: </b>"Andrig Miller" <anmiller@redhat.com><br><b>Cc: </b>"JBoss AS7 Development" <jboss-as7-dev@lists.jboss.org><br><b>Sent: </b>Friday, February 1, 2013 10:13:35 AM<br><b>Subject: </b>Re: [jboss-as7-dev] Removing thread subsystem<br><br><div dir="ltr"><div><div><div><div>Would it be that inconvenient for users that need to use custom executors in web (infinispan,jgroups) to also add thread subsystem configuration?<br><br></div>My original proposal was just to remove the subsystem from shipped configurations, it could still be used if user would need it, which is obvious case for performance tuning web server and clustering.<br>
<br></div>Given that it is not much more work, two lines in xml or two operations in CLI.<br><br></div>As for removal of subsystem itself goes, as David said that is not going to happen for EAP6.x, as we need to support all configurations/subsystems we shipped.<br>
<br>--<br></div>tomaz<br><div><div></div></div></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: 0pt 0pt 0pt 0.8ex; border-left: 1px solid rgb(204, 204, 204); 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>
</blockquote><br></div></body></html>