On Fri, May 27, 2016 at 10:36 PM, Amit Kapila <amit.kapil...@gmail.com> wrote:
> On Sat, May 28, 2016 at 12:28 AM, Andres Freund <and...@anarazel.de> wrote:
>> contain_volatile_functions_walker is duplicated, near entirely, in
>> contain_volatile_functions_not_nextval_walker.
> Previously, I also had same observation.
>> Wouldn't it have been better not to duplicate, and keep a flag about
>> ignoring nextval in the context variable?
> makes sense.  +1 for doing it in the way as you are suggesting.

+1 from me, too.

Robert Haas
EnterpriseDB: http://www.enterprisedb.com
The Enterprise PostgreSQL Company

Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org)
To make changes to your subscription:

Reply via email to