Re: This script will crash the connection

> Bruce Momjian <pgman@candle.pha.pa.us> writes:
> > OK, added to TODO:
> > * Allow NOTIFY in rules
> >> 
> >> Uh, what does that have to do with the problem?  It's certainly not
> >> an accurate rendering of either the current or proposed status ...
> 
> > Oops, can you give me a line.  What was the issue?
> 
> "Allow NOTIFY in conditional rules" would be an approximation.  It's
> not the whole story though, because presently we also have to fail
> if the rule is applied to a query with conditions, even if the rule
> itself is unconditional.  As of my last commit:
> 
> regression=# create rule r1 as on update to int4_tbl do notify foo;
> CREATE
> regression=# update int4_tbl set f1 = f1;
> UPDATE 5
> regression=# update int4_tbl set f1 = f1 where f1 < 0;
> ERROR:  Conditional NOTIFY is not implemented
> 
> which is pretty ugly but at least it doesn't pretend to do something
> it can't, which was the 7.0 behavior.  (In 7.0 you'd have gotten a
> NOTIFY whether the update updated any rows or not.)

Added to TODO:
* Allow NOTIFY in rules involving conditionals  

This covers both cases of conditionals in the rule or the query.

--  Bruce Momjian                        |  http://candle.pha.pa.us pgman@candle.pha.pa.us               |  (610)
853-3000+  If your life is a hard drive,     |  830 Blythe Avenue +  Christ can be your backup.        |  Drexel Hill,
Pennsylvania19026