On 11/18/17 12:05, Tom Lane wrote: > Peter Eisentraut <peter.eisentr...@2ndquadrant.com> writes: >> On 11/17/17 12:16, Tom Lane wrote: >>> I'm confused by the places that change PLy_elog calls to pass NULL: >>> >>> - PLy_elog(ERROR, "could not create globals"); >>> + PLy_elog(ERROR, NULL); >>> >>> How is that an improvement? Otherwise it looks reasonable. > >> If we pass NULL, then the current Python exception becomes the primary >> error, so you'd end up with an "out of memory" error of some kind with a >> stack trace, which seems useful. > > Oh, I see. Objection withdrawn.
Committed, thanks. -- Peter Eisentraut http://www.2ndQuadrant.com/ PostgreSQL Development, 24x7 Support, Remote DBA, Training & Services