Re: pgsql: Skip full index scan during cleanup of B-tree indexes when possi

From: Alexander Korotkov <a(dot)korotkov(at)postgrespro(dot)ru>
To: Teodor Sigaev <teodor(at)sigaev(dot)ru>
Cc: pgsql-committers(at)lists(dot)postgresql(dot)org
Subject: Re: pgsql: Skip full index scan during cleanup of B-tree indexes when possi
Date: 2018-04-04 22:32:36
Message-ID: CAPpHfdvWs3Wf3L=ZOqO29_NPtpsy=hzPMQcK1_=OJ-hXKwNQHw@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-committers

Hi!

On Wed, Apr 4, 2018 at 7:29 PM, Teodor Sigaev <teodor(at)sigaev(dot)ru> wrote:

> Skip full index scan during cleanup of B-tree indexes when possible
>

Thank you for committing this.

It appears that patch contains some redundant variabled. See warnings
produced
by gcc-7.

nbtpage.c: In function '_bt_update_meta_cleanup_info':
nbtpage.c:121:15: warning: variable 'metaopaque' set but not used
[-Wunused-but-set-variable]
BTPageOpaque metaopaque;
^~~~~~~~~~
nbtree.c: In function '_bt_vacuum_needs_cleanup':
nbtree.c:790:15: warning: variable 'metaopaque' set but not used
[-Wunused-but-set-variable]
BTPageOpaque metaopaque;
^~~~~~~~~~

Attached patch fixes this.

------
Alexander Korotkov
Postgres Professional: http://www.postgrespro.com
The Russian Postgres Company

Attachment Content-Type Size
bt-vacuum-cleanup-fix-warnings.patch application/octet-stream 1.4 KB

In response to

Responses

Browse pgsql-committers by date

  From Date Subject
Next Message Peter Geoghegan 2018-04-05 00:00:13 Re: pgsql: Skip full index scan during cleanup of B-tree indexes when possi
Previous Message Simon Riggs 2018-04-04 20:28:03 Re: pgsql: New files for MERGE