Thanks to everyone, fixes are pushed except nodeMerge.c, I don't wish to
increase entropy around MERGE patch :)
Kyotaro HORIGUCHI wrote:
> Hello.
>
> The commit leaves three warnings for
> -Wunused-but-set-variable. Two of them are not assertion-only but
> really not used at all.
>
> I also found that nodeMerge.c has one such variable.
>
> regards.
>
> At Thu, 5 Apr 2018 15:43:55 +0900, Masahiko Sawada <sawada.mshk@gmail.com> wrote in
<CAD21AoAtYzXeuhPqGw89KxqcJWpSKxWRNkYzyJxAqJHZham==Q@mail.gmail.com>
>> On Thu, Apr 5, 2018 at 2:40 PM, Masahiko Sawada <sawada.mshk@gmail.com> wrote:
>>> On Thu, Apr 5, 2018 at 1:30 AM, Teodor Sigaev <teodor@sigaev.ru> wrote:
>>>> Thanks for everyone, pushed with minor editorization
>>>>
>>>
>>> Thank you for committing!
>>> I found a typo in nbtpage.c and attached a patch fixes it.
>>>
>>
>> I also found an incorrect documentation in create_index.sgml as follows.
>>
>> <term><literal>vacuum_cleanup_index_scale_factor</literal></term>
>> <listitem>
>> <para>
>> Per-table value for <xref
>> linkend="guc-vacuum-cleanup-index-scale-factor"/>.
>> </para>
>> </listitem>
>> </varlistentry>
>>
>> I think it should be "Per-index". Attached a patch for fixing it. And
>> sorry for missing it at review.
>>
>> Regards,
>
--
Teodor Sigaev E-mail: teodor@sigaev.ru
WWW: http://www.sigaev.ru/