Hi,
contain_volatile_functions_walker is duplicated, near entirely, in
contain_volatile_functions_not_nextval_walker.
Wouldn't it have been better not to duplicate, and keep a flag about
ignoring nextval in the context variable?
While at it, couldn't we also fold contain_mutable_functions_walker()
together using a similar technique?
Regards,
Andres