You are absolutely right!
Don't know how I slipped that one... and how come testsuite did not catch
this.
Same as for web also goes for infinispan and jgroups.
This 3 subsystems only have msc dependencies to threads subsystem...
I think we can revisit this for AS8, aka make it the same way as jca & ejb3
use this.
--
tomaz
On Fri, Feb 1, 2013 at 5:04 PM, Andrig Miller <anmiller(a)redhat.com> wrote:
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.
This CANNOT BE REMOVED, its absolutely critical.
Andy
------------------------------
*From: *"Tomaž Cerar" <tomaz.cerar(a)gmail.com>
*To: *"JBoss AS7 Development" <jboss-as7-dev(a)lists.jboss.org>
*Sent: *Friday, February 1, 2013 8:42:09 AM
*Subject: *[jboss-as7-dev] Removing thread subsystem
Hi,
Threads subsystem has not been directly used since 7.1.0.CR1.
see
https://issues.jboss.org/browse/AS7-390 for more details.
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.
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.
I would like to remove it from our default shipped configurations as it
does not effect any functionality.
Also I have introduced option to "deprecate" extension, so it now logs
warning when such extension is registered (aka is found in configuration)
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.
It looks like that we removed usage of Threads subsystem in 7.1.0.CR1 but
forgot to remove it from shipped configurations.
my current effort at this can be found at
https://github.com/ctomc/jboss-as/compare/threads
(there is lots of removed redundant & old style tests)
--
tomaz
_______________________________________________
jboss-as7-dev mailing list
jboss-as7-dev(a)lists.jboss.org
https://lists.jboss.org/mailman/listinfo/jboss-as7-dev