Any reason why we should not remove this InterceptorsFactory from ejb3-interceptors-aop.xml AOP advices?
Should have read EJBTHREE-1246, before asking this :) The comments there say:
The ejb3-interceptors can run in multiple modes, two of which are AOP weaved interception (basic) and Container managed interception (direct, proxy and indirect). For EJB lifecycle callbacks to be properly handled in 'basic' mode InterceptorsFactory and InjectInterceptorsFactory are used. Both of which do nothing if used in 'container' mode and will give out a warning as such.
In my opinion, it should not be a WARN message then. Probably a DEBUG would be right.
No, it really means what it says. The construct should not be used, it does nothing anyway. :-)
So try to remove that construct from core and test for regression.
Removing this from the core (ejb3-interceptors-aop.xml) and running the entire testsuite shows no regressions.