pg.ddx.io  pgsql-committers@postgresql.org mailing list archive  
help / color / mirror / Atom feed
From: tgl@postgresql.org
To: pgsql-committers@postgresql.org
Subject: pgsql: Fix SetClientEncoding() to maintain a cache of previously
Date: Thu,  2 Apr 2009 17:30:53 +0000 (UTC)
Message-ID: <20090402173053.CCF96754ADE@cvs.postgresql.org> (raw)

Log Message:
-----------
Fix SetClientEncoding() to maintain a cache of previously selected encoding
conversion functions.  This allows transaction rollback to revert to a
previous client_encoding setting without doing fresh catalog lookups.
I believe that this explains and fixes the recent report of "failed to commit
client_encoding" failures.

This bug is present in 8.3.x, but it doesn't seem prudent to back-patch
the fix, at least not till it's had some time for field testing in HEAD.

In passing, remove SetDefaultClientEncoding(), which was used nowhere.

Modified Files:
--------------
    pgsql/src/backend/utils/mb:
        mbutils.c (r1.82 -> r1.83)
        (http://anoncvs.postgresql.org/cvsweb.cgi/pgsql/src/backend/utils/mb/mbutils.c?r1=1.82&r2=1.83)
    pgsql/src/include/mb:
        pg_wchar.h (r1.87 -> r1.88)
        (http://anoncvs.postgresql.org/cvsweb.cgi/pgsql/src/include/mb/pg_wchar.h?r1=1.87&r2=1.88)




view thread (6+ messages)  latest in thread

Message-ID: <20090402173053.CCF96754ADE@cvs.postgresql.org>
Permalink:  ../20090402173053.CCF96754ADE@cvs.postgresql.org/
Also on:    postgresql.org/message-id/20090402173053.CCF96754ADE@cvs.postgresql.org

 · 

reply

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Reply to all the recipients using the --to and --cc options:
  reply via email

  To: pgsql-committers@postgresql.org
  Cc: tgl@postgresql.org
  Subject: Re: pgsql: Fix SetClientEncoding() to maintain a cache of previously
  In-Reply-To: <20090402173053.CCF96754ADE@cvs.postgresql.org>

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

This inbox is served by DDX for PostgreSQL; see mirroring instructions
for how to clone and mirror all data and code used for this inbox