On Wed, Mar 23, 2016 at 11:01 PM, Tom Lane <[email protected]> wrote: > Michael Paquier <[email protected]> writes: >> That's ugly. We should actually use TextDatumGetCString because the >> index is stored as text here via a Datum, and then it is converted >> back to an integer. So I propose instead the simple patch attached >> that fixes the failure for me. Could you check if that works for you? > > Yeah, this seems better. But that code needs review anyway, as it's using > elog() for user-facing error conditions, and I'm thinking the error texts > could use a bit of attention from a native English speaker.
Thanks. The bug fix is 384dfbd, and the improvements of error messages is ea4b8bd. -- Michael -- Sent via pgsql-hackers mailing list ([email protected]) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers
