Re: [HACKERS] Read-only transactions

2003-01-08 Thread Peter Eisentraut
Tom Lane writes: > case T_VacuumStmt: > /* No XactReadOnly check since this logically changes no data */ > vacuum((VacuumStmt *) parsetree); > break; > > Then it'll be hard to miss the need to think about this when adding a > new statement. Well, I had

Re: [HACKERS] Read-only transactions

2003-01-07 Thread Tom Lane
Peter Eisentraut <[EMAIL PROTECTED]> writes: > Tom Lane writes: >> Where are you planning to check this? > In general, I'm trying to align it like a (self-imposed) permission check. > For the query-like statements I'm looking at ExecCheckRTPerms(). (That > also handles EXECUTE and EXPLAIN most ea

Re: [HACKERS] Read-only transactions

2003-01-07 Thread Peter Eisentraut
Tom Lane writes: > Where are you planning to check this? In general, I'm trying to align it like a (self-imposed) permission check. For the query-like statements I'm looking at ExecCheckRTPerms(). (That also handles EXECUTE and EXPLAIN most easily.) Utility statements have a check in tcop/utili

Re: [HACKERS] Read-only transactions

2003-01-06 Thread Tom Lane
Peter Eisentraut <[EMAIL PROTECTED]> writes: > I would like to implement read-only transactions following the SQL spec, > ... > I think it's light-weight and marginally useful. "Light-weight" would depend on your intended implementation, I suppose. Where are you planning to check this? Also, the

[HACKERS] Read-only transactions

2003-01-06 Thread Peter Eisentraut
I would like to implement read-only transactions following the SQL spec, so we can check off this item on the supported list. According to the list I gathered, the following commands will fail if the transaction is read-only: alter * analyze checkpoint cluster comment create * delete (from non-te