From: | Pavel Stehule <pavel(dot)stehule(at)gmail(dot)com> |
---|---|
To: | Bruce Momjian <bruce(at)momjian(dot)us> |
Cc: | David Fetter <david(at)fetter(dot)org>, PostgreSQL Hackers <pgsql-hackers(at)postgresql(dot)org> |
Subject: | Re: PoC: Make it possible to disallow WHERE-less UPDATE and DELETE |
Date: | 2017-02-02 16:21:03 |
Message-ID: | CAFj8pRB+6uW56Afp_b5sQLMi0-_bCUnbiic=1Arso0giN1Q+ww@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
2017-02-02 16:34 GMT+01:00 Bruce Momjian <bruce(at)momjian(dot)us>:
> On Thu, Feb 2, 2017 at 07:18:45AM -0800, David Fetter wrote:
> > On Thu, Feb 02, 2017 at 03:16:29PM +0000, Bruce Momjian wrote:
> > > I just don't see this patch going in. I think it needs are larger
> > > approach to the problems it is trying to solve. I think it then
> > > will be very useful.
> >
> > What problems that it's trying to solve are not addressed?
>
> Unjoined tables. Inconsistent alias references. I think there are a
> bunch of things and if we can make a list and get a mode to warn about
> all of them, it would be very useful.
>
Identification of unjoined tables should be very useful - but it is far to
original proposal - so it can be solved separately.
This patch is simple - and usually we prefer more simple patches than one
bigger.
Better to enhance this feature step by step.
Regards
Pavel
>
> --
> Bruce Momjian <bruce(at)momjian(dot)us> http://momjian.us
> EnterpriseDB http://enterprisedb.com
>
> + As you are, so once was I. As I am, so you will be. +
> + Ancient Roman grave inscription +
>
>
> --
> Sent via pgsql-hackers mailing list (pgsql-hackers(at)postgresql(dot)org)
> To make changes to your subscription:
> http://www.postgresql.org/mailpref/pgsql-hackers
>
From | Date | Subject | |
---|---|---|---|
Next Message | Konstantin Knizhnik | 2017-02-02 16:27:53 | Re: Deadlock in XLogInsert at AIX |
Previous Message | David Fetter | 2017-02-02 16:04:15 | Re: PoC: Make it possible to disallow WHERE-less UPDATE and DELETE |