Modernizing SQL functions' result type coercions

From: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
To: pgsql-hackers(at)lists(dot)postgresql(dot)org
Cc: Petr Fedorov <petr(dot)fedorov(at)phystech(dot)edu>
Subject: Modernizing SQL functions' result type coercions
Date: 2019-11-27 22:57:10
Views: Raw Message | Whole Thread | Download mbox | Resend email
Lists: pgsql-hackers

I got interested in $SUBJECT as a result of the complaint at [1]
about typmods not being checked/enforced in places where they
reasonably should be. The cause is that executor/functions.c's
check_sql_fn_retval() only worries about types not typmods.
Another thing not to like is that it only supports cases where
the target type is binary-compatible with the source. It's not
great that user-visible semantics depend on an implementation
detail like binary compatibility. (Amusingly, our user docs
ignore this altogether and claim that the types must be identical.)

Hence, the attached patch rearranges things so that we'll allow
any case where the parser's standard coercion logic can find an
assignment-level coercion, including typmod coercion if needed.
In a green field I might've argued for restricting this to
implicit coercions; but since some of the standard binary-compatible
casts are assignment-level, that would risk breaking applications
that work today. It's really safe enough though, just as assignment
coercions are fine in INSERT: there's no possible confusion about
which conversion is appropriate.

This required some adjustments of check_sql_fn_retval's API.
I found that pulling out the determination of the result tupdesc
and making the callers do that was advisable: in most cases, the
caller has more information and can produce a more accurate tupdesc
(eg by calling get_call_result_type not get_func_result_type).
I also pulled out creation of the JunkFilter that functions.c
wants (but none of the other callers do); having it in just one
place seems simpler. A nice side-effect of these changes is that
we can inline SQL functions in some cases where that wasn't
possible before.

This could use review/testing, so I'll add it to the next CF.

regards, tom lane


Attachment Content-Type Size
cast-sql-function-results-properly-1.patch text/x-diff 53.4 KB


Browse pgsql-hackers by date

  From Date Subject
Next Message Jeff Davis 2019-11-27 22:58:04 Re: Memory-Bounded Hash Aggregation
Previous Message Alvaro Herrera 2019-11-27 21:36:50 Re: FETCH FIRST clause WITH TIES option