Re: Error code missing for "wrong length of inner sequence" error

On 01/10/2020 14:21, Daniel Gustafsson wrote:
>> On 1 Oct 2020, at 12:54, Heikki Linnakangas <hlinnaka(at)iki(dot)fi> wrote:
>
>> Most checks when converting between SQL and Python types use the PLy_elog() function, which uses the genericc ERRCODE_EXTERNAL_ROUTINE_EXCEPTION error code, but I think ERRCODE_ARRAY_SUBSCRIPT_ERROR is better.
>
> On that note, wouldn't the dimension check errors in PLySequence_ToArray be
> just as well off using normal ereport()'s? Only one of them seem to error out
> in a way that could propagate an error from Python to postgres.

Yes, you are right. I was going to say that we are not very consistent
on when to use ereport() and when PLy_elog() in general, but looking
closer, we are. Outside the array-conversion functions, PLy_elog() is
only used when a Python C API call fails, and ereport() is used otherwise.

Patch attached.

- Heikki