Re: TODO: Add a GUC to control whether BEGIN inside - Mailing list pgsql-hackers

From Joshua D. Drake
Subject Re: TODO: Add a GUC to control whether BEGIN inside
Date
Msg-id 1167329405.24530.41.camel@localhost.localdomain
Whole thread Raw
In response to Re: TODO: Add a GUC to control whether BEGIN inside  (Bruce Momjian <bruce@momjian.us>)
Responses Re: TODO: Add a GUC to control whether BEGIN inside  (Tom Lane <tgl@sss.pgh.pa.us>)
List pgsql-hackers
>             1
>     (1 row)
>     
>     test=> COMMIT;
>     COMMIT
> 
> I think you can make the case that this should be an error, or at least
> that's how it got on the TODO list.  I can always remove it if people
> don't want the item completed.

Well I can tell you that my customers who are postgresql users ;) would
howl in fury if we did that. They are already significantly irritated
that certain errors are so strict. E.g.,

postgres=# BEGIN;
BEGIN
postgres=# ALTER TABLE baz ADD COLUMN bar text;
ERROR:  relation "baz" does not exist
postgres=# SELECT * FROM foo;
ERROR:  current transaction is aborted, commands ignored until end of
transaction block

You do not need to argue with me about the purpose :), I understand why
it is just really frustrating for many users.

I would say that a GUC variable for such behavior as listed in the TODO
is overzealous. We should either enforce it, or not. As we do not now,
there is no reason imo to change it.

Sincerely,

Joshua D. Drake

-- 
     === The PostgreSQL Company: Command Prompt, Inc. ===
Sales/Support: +1.503.667.4564 || 24x7/Emergency: +1.800.492.2240
Providing the most comprehensive  PostgreSQL solutions since 1997            http://www.commandprompt.com/

Donate to the PostgreSQL Project: http://www.postgresql.org/about/donate





pgsql-hackers by date:

Previous
From: Bruce Momjian
Date:
Subject: Re: Load distributed checkpoint
Next
From: "Joshua D. Drake"
Date:
Subject: Re: TODO: GNU TLS