On Wed, Sep 21, 2016 at 12:32 AM, Robert Haas <[email protected]> wrote: > For what it's worth, I think it's fine. Good error messages are a useful > thing. > > More generally, I think the whole patch looks good and should be committed.
Hm. I'd think that it is still more portable to just issue a WARNING message in palloc_extended() when MCXT_ALLOC_NO_OOM then. Other code paths could benefit from that as well, and the patch proposed does nothing for the other places calling it. I am fine to write a patch for this purpose if you agree on that. -- Michael -- Sent via pgsql-hackers mailing list ([email protected]) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers
