Its a bit confusing since there are two RARDeployment classes
(one that actually is the RARDeployment, the other for the MCF - ManagedConnectionFactory).
I assume you are talking about the one in
which is the one that turns the MCF into an MBean (by wrapping it).
Yes that is the correct place. It controls the MCF "lifecycle".
I believe the main change you need (beside the recovery registration)
is factoring out the security subject creation that is currently in BaseConnectionManager2?
It would be better named MCFDeployment or ConnectionDefinitionDeployment.