This PR changes to configure jOOQ ConnectionProvider via customizer as it seems to be the original intention based on #24732.
Comment From: snicoll
Ah ha. Good catch there @izeye, I forgot that one.
Comment From: snicoll
Actually, let me take that back. I did that on purpose, but didn't do it properly anyway.
The plan with the customizer was to only customise things that have been defined externally. I didn't configure ConnectionProvider this way as the plan is to deprecate the customizer eventually and that would stop configuring the ConnectionProvider that we auto-configure ourselves.
Re-reading the code, I noticed we also auto-configure a DefaultExecuteListenerProvider, so the processing of ExecuteListenerProvider should rather move to the main auto-configuration for the same reason.
Would you be willing to review your PR in that direction?
Comment From: izeye
@snicoll Thanks for the feedback! I missed it. I updated as you suggested.
Comment From: snicoll
Thanks for the quick feedback Johnny! I've polished by adding assertions outside of that deprecated test to make it more obvious those two providers will be honoured once we remove the customizer (see dcc0ca0).