The default constructor for IndirectRecoverableConnection attempts to use a null variable as input to the datasource setup fn. This naturally fails horribly. The default ctor should not attempt the setup, it's not possible until the datasource name is set. The datasource will be inited lazily on demand, so removing the setup attempt from the ctor is not a problem.