On 3 July 2018 at 16:55, Michael Paquier <mich...@paquier.xyz> wrote: > Okay, the patch looks logically correct to me, I just tweaked the > comments as per the attached. I would also back-patch that down to v11 > to keep the code consistent with HEAD.. What do you think?
Thanks for fixing it up. It looks fine apart from "Temporation" should be "Temporary". I think it should be backpatched to v11 and v10. Your original commit went there too. I don't see any reason to do any different here than what you did with the original commit. -- David Rowley http://www.2ndQuadrant.com/ PostgreSQL Development, 24x7 Support, Training & Services
