On Sat, Mar 25, 2017 at 2:42 AM, Fujii Masao <masao.fu...@gmail.com> wrote:
> On Thu, Mar 23, 2017 at 4:28 AM, Fujii Masao <masao.fu...@gmail.com> wrote:
>> On Wed, Mar 15, 2017 at 7:51 PM, Masahiko Sawada <sawada.m...@gmail.com>
>> wrote:
>>> On Wed, Mar 15, 2017 at 1:09 PM, Yugo Nagata <nag...@sraoss.co.jp> wrote:
>>>> On Fri, 10 Mar 2017 20:08:42 +0900
>>>> Masahiko Sawada <sawada.m...@gmail.com> wrote:
>>>>
>>>>> On Fri, Mar 10, 2017 at 3:58 PM, Jim Nasby <jim.na...@openscg.com> wrote:
>>>>> > On 3/6/17 8:34 PM, Masahiko Sawada wrote:
>>>>> >>
>>>>> >> I don't think it can say "1 frozen pages" because the number of
>>>>> >> skipped pages according to visibility map is always more than 32
>>>>> >> (SKIP_PAGES_THRESHOLD).
>>>>> >
>>>>> >
>>>>> > That's just an artifact of how the VM currently works. I'm not a fan of
>>>>> > cross dependencies like that unless there's a pretty good reason.
>>>>>
>>>>> Thank you for the comment.
>>>>> Agreed. Attached fixed version patch.
>>>>
>>>> It seems that it says "frozen pages" if pinskipped_pages is not zero,
>>>> rather than about frozenskipped_pages.
>>>>
>>>> How about writing as below?
>>>>
>>>> appendStringInfo(&buf, ngettext("Skipped %u page due to buffer pins, "
>>>> "Skipped %u pages due to buffer pins,
>>>> ",
>>>> vacrelstats->pinskipped_pages),
>>>> vacrelstats->pinskipped_pages);
>>>> appendStringInfo(&buf, ngettext("%u frozen page.\n", "%u frozen
>>>> pages.\n",
>>>> vacrelstats->frozenskipped_pages),
>>>> vacrelstats->frozenskipped_pages);
>>>>
>>>
>>> Yeah, you're right. I've attached the updated version patch
>>> incorporated your comment and fixing documentation.
>>
>> The patch looks very simple and good to me.
>> Barring objection, I will commit this.
>
> Pushed.
>
Thank you!
Actually, vacuum execution example in doc is still inconsistent with
actual output because commit 9eb344faf54a849898d9be012ddfa8204cfeb57c
added the oldest xmin to the vacuum verbose log. I mentioned another
thread[1] but it's still inconsistent.
[1]
https://www.postgresql.org/message-id/cad21aoaga2pb3p-cwmtkxbsbkzs1bcdgblcyvcvcdxspg_x...@mail.gmail.com
Regards,
--
Masahiko Sawada
NIPPON TELEGRAPH AND TELEPHONE CORPORATION
NTT Open Source Software Center
--
Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-hackers
