Re: Verbosity of column definition mismatch error

From: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
To: Brendan Jurd <blakjak(at)blakjak(dot)sytes(dot)net>
Cc: pgsql-hackers(at)postgresql(dot)org
Subject: Re: Verbosity of column definition mismatch error
Date: 2005-01-10 23:42:43
Message-ID: 9922.1105400563@sss.pgh.pa.us
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

Brendan Jurd <blakjak(at)blakjak(dot)sytes(dot)net> writes:
> ERROR: query-specified return row and actual function return row do not
> match

> With that in mind, I'd like to submit a request, that the verbosity of
> this error message be increased to explain which ordinal column position
> the error occurred at, and, if possible, the details of the mismatch.

This is a fairly trivial change, but unfortunately it's too late for
8.0, because we froze error message texts some time ago (for the benefit
of translators).

If you want to start working on a patch for 8.1, look in
src/backend/executor/nodeFunctionscan.c. I'd be inclined to forget the
bool return convention for tupledesc_match() and just have it ereport
directly when it sees a problem. The rowtype matching code in
check_sql_fn_retval (in src/backend/executor/functions.c) would be
useful as a reference.

regards, tom lane

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Brendan Jurd 2005-01-11 00:02:28 Re: Verbosity of column definition mismatch error
Previous Message David Fetter 2005-01-10 22:30:36 Returning multiple rowsets from a function