Patch for search_path --- apply to 7.4 branch?

Started by Tom Laneover 22 years ago3 messagespatches
Jump to latest
#1Tom Lane
tgl@sss.pgh.pa.us

Attached is the patch I just applied to CVS HEAD to fix the search_path
problem that Olivier Prenant recently identified (can't set a search
path for another database if it mentions schemas not present in the
current database). I am looking for comments on whether to back-patch
this into REL7_4_STABLE. It is a rather large patch to fix what might
be considered a non-critical problem, so I'm a bit hesitant about back
patching. Any opinions?

regards, tom lane

#2Olivier PRENANT
ohp@pyrenet.fr
In reply to: Tom Lane (#1)
Re: Patch for search_path --- apply to 7.4 branch?

Hi Tom,

I'd love to see it in a 7.4.2... With maybe the patch for SMP...

Regards
On Mon, 19 Jan 2004, Tom Lane wrote:

Date: Mon, 19 Jan 2004 14:15:08 -0500
From: Tom Lane <tgl@sss.pgh.pa.us>
To: pgsql-patches@postgreSQL.org
Subject: [PATCHES] Patch for search_path --- apply to 7.4 branch?

Attached is the patch I just applied to CVS HEAD to fix the search_path
problem that Olivier Prenant recently identified (can't set a search
path for another database if it mentions schemas not present in the
current database). I am looking for comments on whether to back-patch
this into REL7_4_STABLE. It is a rather large patch to fix what might
be considered a non-critical problem, so I'm a bit hesitant about back
patching. Any opinions?

regards, tom lane

--
Olivier PRENANT Tel: +33-5-61-50-97-00 (Work)
6, Chemin d'Harraud Turrou +33-5-61-50-97-01 (Fax)
31190 AUTERIVE +33-6-07-63-80-64 (GSM)
FRANCE Email: ohp@pyrenet.fr
------------------------------------------------------------------------------
Make your life a dream, make your dream a reality. (St Exupery)

#3Neil Conway
neilc@samurai.com
In reply to: Tom Lane (#1)
Re: Patch for search_path --- apply to 7.4 branch?

Tom Lane <tgl@sss.pgh.pa.us> writes:

Attached is the patch I just applied to CVS HEAD to fix the
search_path problem that Olivier Prenant recently identified (can't
set a search path for another database if it mentions schemas not
present in the current database). I am looking for comments on
whether to back-patch this into REL7_4_STABLE.

The patch looks pretty safe to me: while it touches a fair number of
places, most of the changes are just search-and-replace stuff, not
actual logic changes. I wouldn't have a problem with this being
applied to the 7.4 branch.

-Neil