Re: Out-of-memory error reports in libpq

"Bossart, Nathan" <bossartn@amazon.com> writes:
> -            appendPQExpBuffer(&conn->errorMessage,
> -                              libpq_gettext("out of memory allocating GSSAPI buffer (%d)\n"),
> -                              payloadlen);
> +            pqReportOOM(conn);

> I see that some context is lost in a few places (e.g., the one above
> points to a GSSAPI buffer).  Perhaps this extra context could be
> useful to identify problematic areas, but it might be unlikely to help
> much in these parts of libpq.  In any case, the vast majority of
> existing callers don't provide any extra context.

Yeah, there are half a dozen places that currently print something
more specific than "out of memory".  I judged that the value of this
was not worth the complexity it'd add to support it in this scheme.
Different opinions welcome of course.

            regards, tom lane