| From: | amul sul <sulamul(at)gmail(dot)com> |
|---|---|
| To: | Andrew Borodin <amborodin(at)acm(dot)org> |
| Cc: | Robert Haas <robertmhaas(at)gmail(dot)com>, Jim Nasby <Jim(dot)Nasby(at)bluetreble(dot)com>, David Fetter <david(at)fetter(dot)org>, Craig Ringer <craig(at)2ndquadrant(dot)com>, PostgreSQL-development <pgsql-hackers(at)postgresql(dot)org> |
| Subject: | Re: pg_background contrib module proposal |
| Date: | 2017-01-07 14:12:35 |
| Message-ID: | CAAJ_b97TgM0ZpWERNfBkoqTm+04V+07ehLRiVvOdjognyU99HQ@mail.gmail.com |
| Views: | Raw Message | Whole Thread | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-hackers |
On Sat, Jan 7, 2017 at 2:06 PM, Andrew Borodin <borodin(at)octonica(dot)com> wrote:
> Hi!
>
> 2017-01-07 11:44 GMT+05:00 amul sul <sulamul(at)gmail(dot)com>:
>
>> Changes:
>> 1. pg_background_launch renamed to pg_background_start
>> 2. pg_background_detach renamed to pg_background_close
>> 3. Added new api to display previously launch background worker & its
>> result count waiting to be read.
> Great work!
>
Thanks.
>
>> Note that attached patches need to apply to the top of the Peter
>> Eisentraut's v2 patch[1].
> I've looked a bit into that patch. I thought you would switch
> MemoryContext before calling BackgroundSessionStart() from
> pg_background? Have I got something wrong?
>
Hmm, understood your point, let pg_background extension do the
required context setup, attached version patch does that change.
Thanks.
Regards,
Amul
| Attachment | Content-Type | Size |
|---|---|---|
| 0001-Changes-to-Peter-Eisentraut-s-bgsession-v2-patch.patch | application/octet-stream | 2.9 KB |
| 0002-pg_background-as-client-of-BackgroundSession-v2.patch | application/octet-stream | 17.7 KB |
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Andrew Gierth | 2017-01-07 15:57:30 | Teaching query_planner to handle multiple sort orders? |
| Previous Message | Greg Stark | 2017-01-07 13:25:08 | Re: pg_stat_activity.waiting_start |