On Wed, Aug 31, 2016 at 9:41 AM, Peter Eisentraut
>> Joy, do you have an idea what a *minimally invasive* patch for C++
>> support would look like? That's certainly the first step here.
> I developed a minimally invasive patch for C++ support a few years ago
> shortly after I wrote that blog post. Since there appears to have been
> some interest here now, I have updated that and split it up into logical
> So here you go.
I'm ready for my first-ever PG code review, and I was considering
tackling this one. I.e., https://commitfest.postgresql.org/10/776/
Could someone help me with a few procedural questions?
(1) This page: https://wiki.postgresql.org/wiki/Reviewing_a_Patch
lists the current commitfest's manager as "(vacant)". But this page:
https://commitfest.postgresql.org/ seems to indicate that a commitfest
is currently in progress. Is there a commitfest manager at the
(2) It seems like there are still a few big questions about this commit:
- Is it wanted at the moment? It didn't seem like there's a
consensus about whether or not this enhancement should be
merged, even if the patch is pretty minimal.
- It seems like there are two competing patch
sets in play for this enhancement: Joy's and
Peter's. Presumably at most one of them would
Are these signs that I should find a different commit to review
for now, so that this one can be further discussed?
Thanks very much,
Sent via pgsql-hackers mailing list (email@example.com)
To make changes to your subscription: