Re: pg_background (and more parallelism infrastructure patches) - Mailing list pgsql-hackers

From Petr Jelinek
Subject Re: pg_background (and more parallelism infrastructure patches)
Date
Msg-id 5411C256.6050103@2ndquadrant.com
Whole thread Raw
In response to Re: pg_background (and more parallelism infrastructure patches)  (Robert Haas <robertmhaas@gmail.com>)
List pgsql-hackers
On 10/09/14 22:53, Robert Haas wrote:
> Here's what the other approach looks like.  I can't really see doing
> this way and then only providing hooks for those two functions, so
> this is with hooks for all the send-side stuff.
>
> Original version: 9 files changed, 295 insertions(+), 3 deletions(-)
> This version: 9 files changed, 428 insertions(+), 47 deletions(-)
>
> There is admittedly a certain elegance to providing a complete set of
> hooks, so maybe this is the way to go.  The remaining patches in the
> patch series work with either the old version or this one; the changes
> here don't affect anything else.
>
> Anyone else have an opinion on which way is better here?

Ok so it is more than 20 lines :)

I do like this approach more though, it looks cleaner to me.


On 10/09/14 22:53, Robert Haas wrote:
> +extern int    (*pq_putmessage_hook)(char msgtype, const char *s, size_tlen);
> +extern int  (*pq_flush_hook)(void);

Btw you forgot to remove the above.

--  Petr Jelinek                  http://www.2ndQuadrant.com/ PostgreSQL Development, 24x7 Support, Training &
Services



pgsql-hackers by date:

Previous
From: Tom Lane
Date:
Subject: Re: Memory Alignment in Postgres
Next
From: Pavel Stehule
Date:
Subject: Re: proposal (9.5) : psql unicode border line styles