Re: Improvement in log message of logical replication worker

From: Masahiko Sawada <sawada(dot)mshk(at)gmail(dot)com>
To: Peter Eisentraut <peter(dot)eisentraut(at)2ndquadrant(dot)com>
Cc: PostgreSQL-development <pgsql-hackers(at)postgresql(dot)org>
Subject: Re: Improvement in log message of logical replication worker
Date: 2017-05-25 01:30:24
Message-ID: CAD21AoArf2ef6G_g=8owQFFetOeascSbt3Fa5VMFUABbAN85zA@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

On Wed, May 24, 2017 at 7:04 PM, Peter Eisentraut
<peter(dot)eisentraut(at)2ndquadrant(dot)com> wrote:
> On 5/17/17 23:36, Masahiko Sawada wrote:
>> The patch looks good to me.
>> There are some log messages saying just 'logical replication worker
>> for subscription ...' in reread_subscription but should we add 'apply'
>> to these messages in order for user to distinguish between apply
>> worker and table sync worker?
>
> done and done
>

Thank you!

Regards,

--
Masahiko Sawada
NIPPON TELEGRAPH AND TELEPHONE CORPORATION
NTT Open Source Software Center

In response to

Browse pgsql-hackers by date

  From Date Subject
Next Message Robert Haas 2017-05-25 01:36:29 Re: pg_dump ignoring information_schema tables which used in Create Publication.
Previous Message Tsunakawa, Takayuki 2017-05-25 00:45:49 Re: [PATCH v1] Add and report the new "in_hot_standby" GUC pseudo-variable.