Postgres Pro
Downloads
Home   >   mailing lists

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

From Alexander Korotkov
Subject Re: pgsql: Skip full index scan during cleanup of B-tree indexes when possi
Date
Msg-id CAPpHfdvWs3Wf3L=ZOqO29_NPtpsy=hzPMQcK1_=OJ-hXKwNQHw@mail.gmail.com
Whole thread Raw
In response to pgsql: Skip full index scan during cleanup of B-tree indexes whenpossi  (Teodor Sigaev <teodor@sigaev.ru>)
Responses Re: pgsql: Skip full index scan during cleanup of B-tree indexes when possi  (Peter Geoghegan <pg@bowt.ie>)
Re: pgsql: Skip full index scan during cleanup of B-tree indexes when possi  (Alexander Korotkov <a.korotkov@postgrespro.ru>)
List pgsql-committers
Tree view
Hi!

On Wed, Apr 4, 2018 at 7:29 PM, Teodor Sigaev <teodor@sigaev.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

pgsql-committers by date:

Previous
From: Tom Lane
Date:
Subject: Re: pgsql: Restore erroneously removed ONLY from PK check
Next
From: Peter Geoghegan
Date:
Subject: Re: pgsql: Skip full index scan during cleanup of B-tree indexes when possi