Re: [PATCH] Re: [pgsql-advocacy] Why READ ONLY transactions?

From: Bruce Momjian <pgman(at)candle(dot)pha(dot)pa(dot)us>
To: Sean Chittenden <sean(at)chittenden(dot)org>
Cc: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>, Josh Berkus <josh(at)agliodbs(dot)com>, pgsql-patches(at)postgresql(dot)org
Subject: Re: [PATCH] Re: [pgsql-advocacy] Why READ ONLY transactions?
Date: 2003-12-01 23:10:46
Message-ID: 200312012310.hB1NAkh11727@candle.pha.pa.us
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-advocacy pgsql-hackers pgsql-patches

Sean Chittenden wrote:
> I think Tom's big objection is the abuse of the GUC system for
> maintaining this information. Having thought about this some, I think
> the GUC system is pretty well suited for this and that Tom's objection
> (correct me if I'm wrong here) is that GUC has a non-hierarchical
> naming structure/convention. With a hierarchical structure, lumping
> of GUC variables becomes more reasonable and the naming is more
> systematic. Instead of, "jail_read_only_transaction=true" it'd be
> "security.force_readonly=true" or "transaction.readonly_always=true".

Agreed on the usefulness of GUC. I had trouble adding security for
logging settings not because GUC wouldn't work but because the logging
control had to hit several different variables that all had different
API's. It had to allow _increase_ for some variables and not others.

--
Bruce Momjian | http://candle.pha.pa.us
pgman(at)candle(dot)pha(dot)pa(dot)us | (610) 359-1001
+ If your life is a hard drive, | 13 Roberts Road
+ Christ can be your backup. | Newtown Square, Pennsylvania 19073

In response to

Browse pgsql-advocacy by date

  From Date Subject
Next Message Tom Lane 2003-12-01 23:17:16 Re: [PATCH] Re: [pgsql-advocacy] Why READ ONLY transactions?
Previous Message Sean Chittenden 2003-12-01 23:07:01 Re: [PATCH] Re: [pgsql-advocacy] Why READ ONLY transactions?

Browse pgsql-hackers by date

  From Date Subject
Next Message Bruce Momjian 2003-12-01 23:15:22 Re: [HACKERS] initdb mkdir_p() doesn't work
Previous Message Sean Chittenden 2003-12-01 23:07:01 Re: [PATCH] Re: [pgsql-advocacy] Why READ ONLY transactions?

Browse pgsql-patches by date

  From Date Subject
Next Message Bruce Momjian 2003-12-01 23:11:53 Re: YA Cygwin DLLIMPORT patch
Previous Message Bruce Momjian 2003-12-01 23:08:35 Re: cleanup execTuples.c