From: | David Steele <david(at)pgmasters(dot)net> |
---|---|
To: | Pavan Deolasee <pavan(dot)deolasee(at)gmail(dot)com> |
Cc: | Masahiko Sawada <sawada(dot)mshk(at)gmail(dot)com>, pgsql-hackers <pgsql-hackers(at)postgresql(dot)org>, Jeff Janes <jeff(dot)janes(at)gmail(dot)com>, Simon Riggs <simon(at)2ndquadrant(dot)com> |
Subject: | Re: Re: COPY FREEZE and setting PD_ALL_VISIBLE/visibility map bits |
Date: | 2019-03-20 10:30:53 |
Message-ID: | ff4c55c4-ac5b-e709-87c2-915cf14221f3@pgmasters.net |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
Hi Pavan,
On 3/14/19 2:20 PM, Masahiko Sawada wrote:
> On Thu, Mar 14, 2019 at 5:17 PM Pavan Deolasee <pavan(dot)deolasee(at)gmail(dot)com> wrote:
>>
>> Ok. I will run some tests. But please note that this patch is a bug fix to address the performance issue that is caused by having to rewrite the entire table when all-visible bit is set on the page during first vacuum. So while we may do some more work during COPY FREEZE, we're saving a lot of page writes during next vacuum. Also, since the scan that we are doing in this patch is done on a page that should be in the buffer cache, we will pay a bit in terms of CPU cost, but not anything in terms of IO cost.
>
> Agreed. I had been misunderstanding this patch. The page scan during
> COPY FREEZE is necessary and it's very cheaper than doing in the first
> vacuum.
I have removed Ibrar as a reviewer since there has been no review from
them in three weeks, and too encourage others to have a look.
Regards,
--
-David
david(at)pgmasters(dot)net
From | Date | Subject | |
---|---|---|---|
Next Message | David Steele | 2019-03-20 10:34:13 | Re: Re: [RFC] [PATCH] Flexible "partition pruning" hook |
Previous Message | David Steele | 2019-03-20 10:15:15 | Re: Re: Reporting script runtimes in pg_regress |