On Fri, Dec 1, 2017 at 11:29 AM, Bossart, Nathan <bossa...@amazon.com> wrote: > Thanks for the review, Robert. I've attached a new version that > addresses your feedback.
Thanks. I think this looks fine now, except that (1) it needs a pgindent run and (2) I vote for putting the test case back. Michael thought the test case was too much because this is so obscure, but I think that's exactly why it needs a test case. Otherwise, somebody a few years from now may not even be able to figure out how to hit this message, and if it gets broken, we won't know. This code seems to be fairly easy to break in subtle ways, so I think more test coverage is good. -- Robert Haas EnterpriseDB: http://www.enterprisedb.com The Enterprise PostgreSQL Company
