Missing RULEs from VIEWs and their format

Lists: pgadmin-hackers
From: Erwin Brandstetter <brandstetter(at)falter(dot)at>
To: pgadmin-hackers(at)postgresql(dot)org
Subject: Missing RULEs from VIEWs and their format
Date: 2010-04-07 17:47:12
Message-ID: 4BBCC520.4070400@falter.at
Views: Raw Message | Whole Thread | Download mbox | Resend email
Lists: pgadmin-hackers

Hi developers!

I have just filed ticket #167 concerning missing RULEs with VIEWs.
On a related but less important note: the formatting of rules might be
improved if was done like triggers are:

-- status quo
CREATE OR REPLACE RULE on_delete AS
ON DELETE TO test DO INSTEAD DELETE FROM test2
WHERE test2.id = old.id;

-- proposal
CREATE OR REPLACE RULE on_delete AS
ON DELETE
TO test
DO INSTEAD
DELETE FROM test2
WHERE test2.id = old.id;

Regards
Erwin


From: Guillaume Lelarge <guillaume(at)lelarge(dot)info>
To: Erwin Brandstetter <brandstetter(at)falter(dot)at>
Cc: pgadmin-hackers(at)postgresql(dot)org
Subject: Re: Missing RULEs from VIEWs and their format
Date: 2010-04-08 20:38:55
Message-ID: 4BBE3EDF.4050600@lelarge.info
Views: Raw Message | Whole Thread | Download mbox | Resend email
Lists: pgadmin-hackers

Le 07/04/2010 19:47, Erwin Brandstetter a écrit :
> [...]
> I have just filed ticket #167 concerning missing RULEs with VIEWs.
> On a related but less important note: the formatting of rules might be
> improved if was done like triggers are:
>
> -- status quo
> CREATE OR REPLACE RULE on_delete AS
> ON DELETE TO test DO INSTEAD DELETE FROM test2
> WHERE test2.id = old.id;
>
> -- proposal
> CREATE OR REPLACE RULE on_delete AS
> ON DELETE
> TO test
> DO INSTEAD
> DELETE FROM test2
> WHERE test2.id = old.id;
>

Hummm, /me feels ashamed but happy to see you came back with so many bug
reports and feature requests :)

--
Guillaume.
http://www.postgresqlfr.org
http://dalibo.com


From: Erwin Brandstetter <brandstetter(at)falter(dot)at>
To: guillaume(at)lelarge(dot)info
Cc: pgadmin-hackers(at)postgresql(dot)org
Subject: Re: Missing RULEs from VIEWs and their format
Date: 2010-04-09 13:03:10
Message-ID: 4BBF258E.1090209@falter.at
Views: Raw Message | Whole Thread | Download mbox | Resend email
Lists: pgadmin-hackers

On 08.04.2010 22:38, guillaume(at)lelarge(dot)info wrote:
> Le 07/04/2010 19:47, Erwin Brandstetter a écrit :
>
>> [...]
>> I have just filed ticket #167 concerning missing RULEs with VIEWs.
>> On a related but less important note: the formatting of rules might be
>> improved if was done like triggers are:
>>
>> -- status quo
>> CREATE OR REPLACE RULE on_delete AS
>> ON DELETE TO test DO INSTEAD DELETE FROM test2
>> WHERE test2.id = old.id;
>>
>> -- proposal
>> CREATE OR REPLACE RULE on_delete AS
>> ON DELETE
>> TO test
>> DO INSTEAD
>> DELETE FROM test2
>> WHERE test2.id = old.id;
>>
>>
> Hummm, /me feels ashamed but happy to see you came back with so many bug
> reports and feature requests :)
>

Hey, Guillaume, I have made notes over the last couple of months, but
never got around to report - for all the work piling on my desktop.
And I do try to verify my findings before reporting. Wouldn't be much
help, if I provided noise instead of insight, right?
So, the accumulation of reports is much rather a sign of bad timing on
my side than of quality issues.

Regards
Erwin


From: Guillaume Lelarge <guillaume(at)lelarge(dot)info>
To: Erwin Brandstetter <brandstetter(at)falter(dot)at>
Cc: pgadmin-hackers(at)postgresql(dot)org
Subject: Re: Missing RULEs from VIEWs and their format
Date: 2010-04-10 08:09:11
Message-ID: 4BC03227.6080600@lelarge.info
Views: Raw Message | Whole Thread | Download mbox | Resend email
Lists: pgadmin-hackers

Le 09/04/2010 15:03, Erwin Brandstetter a écrit :
> On 08.04.2010 22:38, guillaume(at)lelarge(dot)info wrote:
>> Le 07/04/2010 19:47, Erwin Brandstetter a écrit :
>>
>>> [...]
>>> I have just filed ticket #167 concerning missing RULEs with VIEWs.
>>> On a related but less important note: the formatting of rules might be
>>> improved if was done like triggers are:
>>>
>>> -- status quo
>>> CREATE OR REPLACE RULE on_delete AS
>>> ON DELETE TO test DO INSTEAD DELETE FROM test2
>>> WHERE test2.id = old.id;
>>>
>>> -- proposal
>>> CREATE OR REPLACE RULE on_delete AS
>>> ON DELETE
>>> TO test
>>> DO INSTEAD
>>> DELETE FROM test2
>>> WHERE test2.id = old.id;
>>>
>>>
>> Hummm, /me feels ashamed but happy to see you came back with so many bug
>> reports and feature requests :)
>>
>
> Hey, Guillaume, I have made notes over the last couple of months, but
> never got around to report - for all the work piling on my desktop.

Oh, great.

> And I do try to verify my findings before reporting. Wouldn't be much
> help, if I provided noise instead of insight, right?

Right :)

> So, the accumulation of reports is much rather a sign of bad timing on
> my side than of quality issues.
>

No problem.

Anyways, this is fixed. Thanks for your report.

--
Guillaume.
http://www.postgresqlfr.org
http://dalibo.com