On Wed, Jan 17, 2018 at 3:37 PM, Tom Lane <t...@sss.pgh.pa.us> wrote: > Robert Haas <robertmh...@gmail.com> writes: >> Thanks for the review. Updated patch attached. > > Looks OK to me. Would it be worth annotating the added regression test > case with a comment that this once caused EPQ-related planning problems?
I tend to think somebody who is curious about the origin of any particular test can just use 'git blame' and/or 'git log -Gwhatever' to figure out which commits added it, and that therefore it's not worth including that in the comment explicitly. But I don't care deeply. -- Robert Haas EnterpriseDB: http://www.enterprisedb.com The Enterprise PostgreSQL Company
