Re: async notification patch for dblink

From: "Marcus Kempe" <marcus(dot)kempe(at)gmail(dot)com>
To: "Bruce Momjian" <bruce(at)momjian(dot)us>
Cc: pgsql-hackers(at)postgresql(dot)org, "Joe Conway" <mail(at)joeconway(dot)com>
Subject: Re: async notification patch for dblink
Date: 2009-01-15 09:19:56
Message-ID: 6534f7ae0901150119v54e101b2qeccb5a22f0114bfd@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

Hi,

status of the patch is that it's working fine / as expected.

As is the regression test, my only concern there is if it's testing the
functionality thoroughly enough. But at it's current state I suppose it's in
line with the rest of the regression tests for dblink functionality.

Also, please advice if I'm expected to take any additional actions as part
of submitting this patch.

Best regards,

Marcus Kempe

On Wed, Jan 14, 2009 at 23:12, Bruce Momjian <bruce(at)momjian(dot)us> wrote:

>
> What is the status on this?
>
> ---------------------------------------------------------------------------
>
> Marcus Kempe wrote:
> > This patch adds the ability to retrieve async notifications using dblink,
> > via the addition of the function dblink_get_notify.
> >
> > It is written against cvs head, includes documentation and regression
> > testing. It compiles, tests and works well.
> >
> > I would be interested in some feedback on the regression test.
> > My initial thought was to test the function as thoroughly as possible. So
> I
> > perform listen and notify commands within the test to be able to test all
> > aspects of the code. Even though this works well for me, I get the
> feeling
> > that this is not the correct way to do it. I can find no other testing of
> > the listen/notify functionality in the regression tests, and I imagine
> this
> > is for good reason.
> > If someone would care to explain this, and maybe give a hint about what
> > amount of testing is appropriate for this fairly trivial patch, it would
> be
> > appreciated.
> >
> > Best regards,
> >
> > Marcus Kempe
>
> [ Attachment, skipping... ]
>
> >
> > --
> > 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
>
> --
> Bruce Momjian <bruce(at)momjian(dot)us> http://momjian.us
> EnterpriseDB http://enterprisedb.com
>
> + If your life is a hard drive, Christ can be your backup. +
>

In response to

Browse pgsql-hackers by date

  From Date Subject
Next Message Richard Huxton 2009-01-15 09:47:41 Re: Autovacuum daemon terminated by signal 11
Previous Message Albe Laurenz 2009-01-15 07:41:46 Re: fire trigger for a row without update?