Re: Clean up some elog messages and comments for do_pg_stop_backup and do_pg_start_backup

From: Michael Paquier <michael(at)paquier(dot)xyz>
To: Peter Eisentraut <peter(dot)eisentraut(at)2ndquadrant(dot)com>
Cc: Alvaro Herrera <alvherre(at)2ndquadrant(dot)com>, Postgres hackers <pgsql-hackers(at)postgresql(dot)org>
Subject: Re: Clean up some elog messages and comments for do_pg_stop_backup and do_pg_start_backup
Date: 2018-12-30 05:50:46
Message-ID: 20181230055046.GC608@paquier.xyz
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

On Sat, Dec 29, 2018 at 04:29:28PM +0100, Peter Eisentraut wrote:
> On 22/12/2018 00:42, Michael Paquier wrote:
>> What about the following then? This is your second proposal except
>> that the sentence refers to the backup current running using "this",
>> which shows better the context in my opinion:
>> "This backup can be canceled safely, but it will not be usable without
>> all the WAL segments.
>
> To much emphasis on the "this" I think, implying that there are other
> backups that cannot be canceled safely.
>
> How about "You can safely cancel this backup, ...".

I can live with that, please find an updated patch.

A personal note on the matter: I tend to prefer using the passive form
in such log messages because they are impersonal, and not use the
direct form because it becomes more personally addressed to the user.
I may be living abroad for too long though ;)
--
Michael

Attachment Content-Type Size
func-message-clean-v4.patch text/x-diff 2.5 KB

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Michael Paquier 2018-12-30 05:51:37 Re: plpgsql plugin - stmt_beg/end is not called for top level block of statements
Previous Message Michael Paquier 2018-12-30 05:45:28 Re: Clean up some elog messages and comments for do_pg_stop_backup and do_pg_start_backup