On Jan 3, 2010, at 11:54 AM, Andrew Dunstan wrote:

> There are two issues with this patch. First, how far if at all should it be 
> backpatched? All the way, or 8.3, where we tightened the encoding rules, or 
> not at all?

8.3 seems reasonable.

> Second, It produces errors like this:
> 
>   andrew=# select 'a' || invalid_utf_seq() || 'b';
>   ERROR:  invalid byte sequence for encoding "UTF8": 0xd0
>   HINT:  This error can also happen if the byte sequence does not
>   match the encoding expected by the server, which is controlled by
>   "client_encoding".
>   CONTEXT:  PL/Perl function "invalid_utf_seq"
>   andrew=#
> 
> 
> That hint seems rather misleading. I'm not sure what we can do about it 
> though. If we set the noError param on pg_verifymbstr() we would miss the 
> error message that actually identified the bad data, so that doesn't seem 
> like a good plan.

I'm sure I'm just revealing my ignorance here, but how is the hint misleading?

Best,

David


-- 
Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-hackers

Reply via email to