Re: Diff/Patch integration -> SQL cvs clone

From: Bruce Momjian <pgman(at)candle(dot)pha(dot)pa(dot)us>
To: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
Cc: Peter Eisentraut <peter_e(at)gmx(dot)net>, Jean-Michel POURE <jm(dot)poure(at)freesurf(dot)fr>, PostgreSQL Development <pgsql-hackers(at)postgresql(dot)org>
Subject: Re: Diff/Patch integration -> SQL cvs clone
Date: 2001-11-22 03:44:38
Message-ID: 200111220344.fAM3ic811924@candle.pha.pa.us
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgadmin-hackers pgsql-hackers

> Bruce Momjian <pgman(at)candle(dot)pha(dot)pa(dot)us> writes:
> > Peter, should plsh be added to our supplied server-side programming
> > languages? Seems like a major feature to me and to others as well.
>
> While plsh is surely a cool hack, I've got considerable unease about
> making it into an officially supported feature. The only reasons I can
> see for wanting to use it (over plpgsql, pltcl, plperl, etc) are
> inherently violations of transaction semantics. Who's going to roll
> back your sendmail call when the calling transaction later aborts?
> What's going to ensure that the order of external effects has something
> to do with the serialization order that the database assigns to several
> concurrent transactions?
>
> IMHO plsh is a great tool for shooting yourself in the foot (with a
> large-gauge firearm, in fact). People who know what they're doing are
> welcome to use it ... but those sorts of people can find and install
> it for themselves.
>
> Of course the same worries apply to the untrusted variants of pltcl,
> plperl, etc. Perhaps we need to document the risks they carry. But
> plsh hasn't even got the possibility of a trusted variant :-(

Yes, I see your point, but if you need to do something in the operating
system, like send mail, you are stuck with leaving transactions
semantics and security anyway. If they have to leave those anyway, why
not give them simple shell scripts?

--
Bruce Momjian | http://candle.pha.pa.us
pgman(at)candle(dot)pha(dot)pa(dot)us | (610) 853-3000
+ If your life is a hard drive, | 830 Blythe Avenue
+ Christ can be your backup. | Drexel Hill, Pennsylvania 19026

In response to

Responses

Browse pgadmin-hackers by date

  From Date Subject
Next Message Jean-Michel POURE 2001-11-22 11:01:54 ALTER OR REPLACE feature
Previous Message Tom Lane 2001-11-22 03:36:52 Re: Diff/Patch integration -> SQL cvs clone

Browse pgsql-hackers by date

  From Date Subject
Next Message Bruce Momjian 2001-11-22 03:48:55 Re: Plpython crashing the backend in one easy step
Previous Message Tom Lane 2001-11-22 03:36:52 Re: Diff/Patch integration -> SQL cvs clone