We need to migrate this thread config into the web subsystem somehow,
maybe for the next rev of the subsystem, in place of the executor
option. I'll add that as a task to the JIRA.
Either way we definitely won't remove anything... not for a very long
time anyway...
On 02/01/2013 10:04 AM, Andrig Miller 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.
ThisCANNOT 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
_______________________________________________
jboss-as7-dev mailing list
jboss-as7-dev(a)lists.jboss.org
https://lists.jboss.org/mailman/listinfo/jboss-as7-dev