Re: Optional message to user when terminating/cancelling backend

From: Daniel Gustafsson <daniel(at)yesql(dot)se>
To: Thomas Munro <thomas(dot)munro(at)enterprisedb(dot)com>
Cc: Yugo Nagata <nagata(at)sraoss(dot)co(dot)jp>, Michael Paquier <michael(dot)paquier(at)gmail(dot)com>, PostgreSQL mailing lists <pgsql-hackers(at)postgresql(dot)org>
Subject: Re: Optional message to user when terminating/cancelling backend
Date: 2017-09-03 20:47:10
Message-ID: 566814F6-401B-42AB-A6BE-65EB2F6DA1D3@yesql.se
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

> On 02 Sep 2017, at 02:21, Thomas Munro <thomas(dot)munro(at)enterprisedb(dot)com> wrote:
>
> On Fri, Jun 23, 2017 at 1:48 AM, Daniel Gustafsson <daniel(at)yesql(dot)se> wrote:
>> Good point. I’ve attached a new version which issues a NOTICE on truncation
>> and also addresses all other comments so far in this thread. Thanks a lot for
>> the early patch reviews!
>
> FYI this no longer builds because commit 81c5e46c490e just stole your OIDs:
>
> make[3]: Entering directory
> `/home/travis/build/postgresql-cfbot/postgresql/src/backend/catalog'
> cd ../../../src/include/catalog && '/usr/bin/perl' ./duplicate_oids
> 772
> 972
> make[3]: *** [postgres.bki] Error 1

Thanks, I hadn’t spotted that yet. Attached is an updated patch using new OIDs
to make it compile again.

cheers ./daniel

Attachment Content-Type Size
terminate_msg_v4.patch application/octet-stream 17.2 KB

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Daniel Gustafsson 2017-09-03 20:50:09 Re: adding the commit to a patch's thread
Previous Message MauMau 2017-09-03 20:14:53 Re: [RFC] What would be difficult to make data models pluggable for making PostgreSQL a multi-model database?