Re: [HACKERS] Consistently catch errors from Python _New() functions - Mailing list pgsql-hackers

From Tom Lane
Subject Re: [HACKERS] Consistently catch errors from Python _New() functions
Date
Msg-id 9477.1510939000@sss.pgh.pa.us
Whole thread Raw
In response to [HACKERS] Consistently catch errors from Python _New() functions  (Peter Eisentraut <peter.eisentraut@2ndquadrant.com>)
Responses Re: [HACKERS] Consistently catch errors from Python _New() functions
List pgsql-hackers
Peter Eisentraut <peter.eisentraut@2ndquadrant.com> writes:
> While reviewing some unrelated code, I noticed that we are handling
> error conditions from Python API functions such as PyList_New() and
> PyDict_New() in pretty random ways or not at all.  Here is a patch to
> fix that.

This needs some adjustment in the wake of 687f096ea.

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.
        regards, tom lane


pgsql-hackers by date:

Previous
From: Tom Lane
Date:
Subject: Re: Fix number skipping in to_number
Next
From: Fujii Masao
Date:
Subject: Re: Speed up the removal of WAL files