[jboss-as7-dev] Removing thread subsystem

[Tomaž Cerar]

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 at 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 at gmail.com>
> *To: *"JBoss AS7 Development" <jboss-as7-dev at 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 at lists.jboss.org
> https://lists.jboss.org/mailman/listinfo/jboss-as7-dev
>
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://lists.jboss.org/pipermail/jboss-as7-dev/attachments/20130201/873780fd/attachment.html