@Async aspect should clear cached executors when different BeanFactory set

[cdalexndr]

When using AspectJ and multi-context (ex: testing), the bean factory needs to be changed on the singleton aspect.
This fix clears cached method executors when changing bean factory.
If the first context was previously closed with the executors, then an error will be thrown when trying to reuse the closed executor.

[sbrannen]

@cdalexndr, is there an existing issue associated with this "fix"?

If not, can you please add more information to this PR's description to explain the rationale and what error cases the proposed change fixes?

[cdalexndr]

@sbrannen updated description

[sbrannen]

Related Issues

• TestContext framework should support one AspectJ instance per ApplicationContext [SPR-6353] #11019
• Transaction is not started when two spring contexts are created during tests using AspectJ load time weaving [SPR-7238] #11897
• AnnotationTransactionAspect retains reference to JpaTransactionManager from closed context [SPR-12518] #17123
• Dependency injection of @Configurable objects should work across test suites [SPR-6121] #10789
• AnnotationTransactionAspect retains reference to closed BeanFactory [SPR-7964] #12619

[jhoeller]

While I do not see the need for a DisposableBean implementation as the PR suggests, we can simply clear the cached executors in setBeanFactory directly (for scenarios where the instance is reused). I'll resolve it along those lines.

Thanks for the PR, in any case!