Yeah, looks good.
I think the only question is really if we should keep the ManagedConnectionPool level policies and instead always flush on a pool level, as the management operation would.
There are pros/cons of each solution, so lets keep it as IronJacamar 1.x for now, unless somebody else thinks otherwise.