Re: Segmentation fault using digest from pg_crypto

From: Bruce Momjian <bruce(at)momjian(dot)us>
To: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
Cc: Marko Kreen <markokr(at)gmail(dot)com>, Alvaro Herrera <alvherre(at)commandprompt(dot)com>, Manuel Sugawara <masm(at)fciencias(dot)unam(dot)mx>, pgsql-hackers(at)postgresql(dot)org
Subject: Re: Segmentation fault using digest from pg_crypto
Date: 2007-11-11 03:45:18
Message-ID: 200711110345.lAB3jIv15903@momjian.us
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

Tom Lane wrote:
> Bruce Momjian <bruce(at)momjian(dot)us> writes:
> > Tom Lane wrote:
> >> If you don't re-run a contrib module's sql script then you don't get any
> >> new functions that may have been added to the module. So I think the
> >> real issue here is that we ought to deprecate the idea of skipping that
> >> step, period.
>
> > The good news is that the scripts have CREATE OR REPLACE function so it
> > would clean up everything to match the library you just installed.
>
> Hmm ... now that you mention it: some of them do, some of them don't.
> And some of them wrap the whole script in BEGIN/COMMIT, which means that
> a conflict against existing definitions would prevent any new
> definitions from being added.
>
> Seems like maybe we need an explicit project policy that contrib scripts
> should be able to overwrite an existing older installation properly;
> maybe with some error reports, but not failing outright.
>
> If so, someone will have to do the legwork of really making that happen
> ... any volunteers?

I have cleaned up the contrib install/uninstall to be more consistent,
used CREATE OR REPLACE function consisently, and removed transaction
blocks.

--
Bruce Momjian <bruce(at)momjian(dot)us> http://momjian.us
EnterpriseDB http://postgres.enterprisedb.com

+ If your life is a hard drive, Christ can be your backup. +

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Bruce Momjian 2007-11-11 03:45:48 Re: Segmentation fault using digest from pg_crypto
Previous Message Tom Lane 2007-11-11 01:22:06 Refactor parse_type.c's lookup API?