From: | Alvaro Herrera <alvherre(at)2ndquadrant(dot)com> |
---|---|
To: | Michael Paquier <michael(at)paquier(dot)xyz> |
Cc: | pgsql-committers(at)lists(dot)postgresql(dot)org |
Subject: | Re: pgsql: Propagate trigger arguments to partitions |
Date: | 2019-07-10 12:53:36 |
Message-ID: | 20190710125336.GA23834@alvherre.pgsql |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-committers |
On 2019-Jul-10, Michael Paquier wrote:
Hello
> This commit has broken the buildfarm on REL_11_STABLE for a couple of
> animals:
> https://buildfarm.postgresql.org/cgi-bin/show_log.pl?nm=prion&dt=2019-07-10%2006%3A27%3A02
Argh. You say "a couple", you mean "half the buildfarm for that branch"
:-(
> error: ‘for’ loop initial declarations are only allowed in C99 mode
> for (int i = 0; i < trigForm->tgnargs; i++)
>
> This gets fixed with the attached. I would personally keep the code
> consistent for v12 and HEAD for this code path, but I am fine to let
> you decide the way you prefer.
I kinda like block-scoped variables, braindead though they are in
C, so I'll just patch pg11. It's not like this code changes too much
anyway (famous last words).
Thanks for pinging
--
Álvaro Herrera https://www.2ndQuadrant.com/
PostgreSQL Development, 24x7 Support, Remote DBA, Training & Services
From | Date | Subject | |
---|---|---|---|
Next Message | Michael Paquier | 2019-07-10 13:11:56 | Re: pgsql: Propagate trigger arguments to partitions |
Previous Message | Alexander Korotkov | 2019-07-10 12:51:03 | pgsql: Assorted fixes for jsonpath documentation |