Re: [HACKERS] Proposed fix for NOTIFY performance degradation

2011-04-24 Thread Simon Riggs
On Sat, Apr 23, 2011 at 8:44 PM, Tom Lane wrote: > Gianni Ciolli writes: >> [ proposes lobotomization of duplicate-elimination behavior in NOTIFY ] > > I think this change is likely to be penny-wise and pound-foolish. > The reason the duplicate check is in there is that things like triggers > may

Re: [HACKERS] Proposed fix for NOTIFY performance degradation

2011-04-24 Thread Simon Riggs
On Sat, Apr 23, 2011 at 2:57 PM, Gianni Ciolli wrote: > * "avg_usec" denotes the average time in microseconds required by each >  NOTIFY statement. > > h2. Test 1 - PostgreSQL HEAD > >   n   |   m   | iter | avg_usec > ---+---+--+-- >    10 |     1 |   10 |   43.730 >   100 |

Re: [HACKERS] Proposed fix for NOTIFY performance degradation

2011-04-23 Thread Tom Lane
Gianni Ciolli writes: > [ proposes lobotomization of duplicate-elimination behavior in NOTIFY ] I think this change is likely to be penny-wise and pound-foolish. The reason the duplicate check is in there is that things like triggers may just do "NOTIFY my_table_changed". If the trigger is fired

[HACKERS] Proposed fix for NOTIFY performance degradation

2011-04-23 Thread Gianni Ciolli
Hi, while measuring NOTIFY execution time, I noticed a significant performance drop. Please find a patch attached, together with some tests; more details are shown below. Best regards, Dr. Gianni Ciolli - 2ndQuadrant Italia PostgreSQL Training, Services and Support gianni.cio...@2ndquadrant.it |