Details
-
Bug
-
Resolution: Not a Bug
-
Major
-
None
-
21.0.0.Final
-
None
-
Undefined
-
---
-
---
Description
The current org.jboss.as.ee.concurrent.ManagedExecutorServiceImpl extends org.glassfish.enterprise.concurrent.ManagedExecutorServiceImpl to take advantage out of the provided implementation. Regarding to the API doc, it would be necessary to extend org.glassfish.enterprise.concurrent.AbstractManagedExecutorServiceAdapter instead. This would protect the container thread pools from lifecycle calls of user code. But it would lack the implementation of o.g.e.c.ManagedExecutorServiceImpl.
I suggest to adopt the o.g.e.c.AbstractManagedExecutorServiceAdapter methods to the o.j.a.e.c.ManagedExecutorServiceImpl.
If not, it would be good to have an API doc explaining why it is OK to call lifecycle methods from user code for the WildFly implementation.