Re: Revised Patch to allow multiple table locks in "Unison"

From: Fernando Nasser <fnasser(at)cygnus(dot)com>
To: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
Cc: Bruce Momjian <pgman(at)candle(dot)pha(dot)pa(dot)us>, Neil Padgett <npadgett(at)redhat(dot)com>, "pgsql-patches(at)postgresql(dot)org" <pgsql-patches(at)postgresql(dot)org>
Subject: Re: Revised Patch to allow multiple table locks in "Unison"
Date: 2001-08-02 23:58:25
Message-ID: 3B69E921.819C316F@cygnus.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers pgsql-patches

Tom Lane wrote:
>
> Fernando Nasser <fnasser(at)cygnus(dot)com> writes:
> > BTW, it seems we have a SQL non-conformance issue here: views that are
> > only projections+selections of a single base table are SQL-updatable.
>
> Indeed. In Postgres terms I think this means that if a CREATE VIEW
> describes a view that meets the spec's constraints to be "updatable",
> we should automatically create a default set of insert/update/delete
> rules for it. This is (or should be) on the TODO list.
>

Agreed.

We should also emit an error if someone tries to update a non-updatable view
(i.e., it is a view and there is no user defined rules for that update operation).
Silently ignoring the update scares me and I bet it is not what the standard
would tell us to do. Any suggestion on how can we do this? I thought of
adding default rules for those cases so they generate the error. But we would
need an error() function or something to invoke from there.

--
Fernando Nasser
Red Hat - Toronto E-Mail: fnasser(at)redhat(dot)com
2323 Yonge Street, Suite #300
Toronto, Ontario M4P 2C9

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Tom Lane 2001-08-03 00:11:37 Rules for updatable views (was Re: [PATCHES] Revised Patch to allow multiple table locks in "Unison")
Previous Message Tom Lane 2001-08-02 23:51:31 Re: Name for new VACUUM

Browse pgsql-patches by date

  From Date Subject
Next Message Tom Lane 2001-08-03 00:11:37 Rules for updatable views (was Re: [PATCHES] Revised Patch to allow multiple table locks in "Unison")
Previous Message Hiroshi Inoue 2001-08-02 23:53:25 Re: ODBC Boolean handling