On 4/22/19 9:01 PM, Michael Paquier wrote:
>
> Also, the extra SELECT query with regexp_replace() is a bit overkill
> for the purpose, and copying again a copy of the regexp around is no
> fun.
>
> In short, I would simplify things as the attached. What do you think?
Outside of the strlen comments, I do like the changes to the tests, +1
I did purposely keep the SCRAM + MD5 changes as separate patches as I
felt they were both atomic, but I have no strong opinions on that.
Thanks,
Jonathan