Re: Re: [PATCHES] Patch to support transactions with BLOBs for current CVS

From: Bruce Momjian <pgman(at)candle(dot)pha(dot)pa(dot)us>
To: Bruce Momjian <pgman(at)candle(dot)pha(dot)pa(dot)us>
Cc: Denis Perchine <dyp(at)perchine(dot)com>, PostgreSQL-patches <pgsql-patches(at)postgresql(dot)org>, PostgreSQL-development <pgsql-hackers(at)postgresql(dot)org>
Subject: Re: Re: [PATCHES] Patch to support transactions with BLOBs for current CVS
Date: 2001-01-21 03:58:40
Message-ID: 200101210358.WAA03449@candle.pha.pa.us
Views: Whole Thread | Raw Message | Download mbox | Resend email
Thread:
Lists: pgsql-hackers pgsql-patches

Sorry, here is a clean version of the patch.

> > http://www.postgresql.org/mhonarc/pgsql-patches/2000-11/msg00013.html
>
> Can people comment on the following patch that Dennis says is needed?
> It prevents BLOB operations outside transactions. Dennis, can you
> explain why BLOB operations have to be done inside transactions?
>
> ---------------------------------------------------------------------------
> Hello,
>
> here is the patch attached which do check in each BLOB operation, if we are
> in transaction, and raise an error otherwise. This will prevent such
> mistakes.
>

--
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

Attachment Content-Type Size
unknown_filename text/plain 2.3 KB

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Tom Lane 2001-01-21 04:39:52 Re: Re: [PATCHES] Patch to support transactions with BLOBs for current CVS
Previous Message Bruce Momjian 2001-01-21 03:51:23 Re: Re: [PATCHES] Patch to support transactions with BLOBs for current CVS

Browse pgsql-patches by date

  From Date Subject
Next Message Tom Lane 2001-01-21 04:39:52 Re: Re: [PATCHES] Patch to support transactions with BLOBs for current CVS
Previous Message Bruce Momjian 2001-01-21 03:51:23 Re: Re: [PATCHES] Patch to support transactions with BLOBs for current CVS