Re: [bug fix] Stats collector is not restarted on the standby

From: Kuntal Ghosh <kuntalghosh(dot)2007(at)gmail(dot)com>
To: Michael Paquier <michael(dot)paquier(at)gmail(dot)com>
Cc: "Tsunakawa, Takayuki" <tsunakawa(dot)takay(at)jp(dot)fujitsu(dot)com>, "pgsql-hackers(at)postgresql(dot)org" <pgsql-hackers(at)postgresql(dot)org>
Subject: Re: [bug fix] Stats collector is not restarted on the standby
Date: 2016-10-26 10:12:43
Message-ID: CAGz5QCL3L-_N8_SshjpywaENeyEc-GcetTBBxvr4mA4YY42bpg@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

On Wed, Oct 26, 2016 at 12:10 PM, Michael Paquier
<michael(dot)paquier(at)gmail(dot)com> wrote:
> On Wed, Oct 26, 2016 at 2:46 PM, Tsunakawa, Takayuki
> <tsunakawa(dot)takay(at)jp(dot)fujitsu(dot)com> wrote:
>> If the stats collector is forcibly terminated on the standby in streaming replication configuration, it won't be restarted until the standby is promoted to the primary. The attached patch restarts the stats collector on the standby.
>>
>> FYI, when the stats collector is down, SELECTs against the statistics views get stale data with the following message.
>>
>> LOG: using stale statistics instead of current ones because stats collector is not responding
>> STATEMENT: select * from pg_stat_user_tables
>
> Oops. This could be a problem for some applications... As far as I can
> see and after playing with it, your patch looks correct.
> --
I've tested with the patch. The patch doesn't solve the problem
completely. In standby, after forcible termination, statistics
collector process is taking some time to get restarted. In between, if
somebody SELECTs against the statistics views, he will still get stale
data with the above LOG message.

--
Thanks & Regards,
Kuntal Ghosh
EnterpriseDB: http://www.enterprisedb.com

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Thomas Munro 2016-10-26 10:34:35 Measuring replay lag
Previous Message Amit Kapila 2016-10-26 10:12:32 Re: Declarative partitioning - another take