On Thu, Feb 4, 2021 at 11:10 AM Mark Dilger <mark.dil...@enterprisedb.com> wrote: > Numbered 0001 in this next patch set.
Hi, I committed 0001 as you had it and 0002 with some more cleanups. Things I did: - Adjusted some comments. - Changed processQueryResult so that it didn't do foo(bar) with foo being a pointer. Generally we prefer (*foo)(bar) when it can be confused with a direct function call, but wunk->foo(bar) is also considered acceptable. - Changed the return type of ParallelSlotResultHandler to be bool, because having it return PGresult * seemed to offer no advantages. -- Robert Haas EDB: http://www.enterprisedb.com
