Re: zheap: a new storage format for PostgreSQL

From: Daniel Westermann <daniel(dot)westermann(at)dbi-services(dot)com>
To: Tomas Vondra <tomas(dot)vondra(at)2ndquadrant(dot)com>, Amit Kapila <amit(dot)kapila16(at)gmail(dot)com>
Cc: PostgreSQL Hackers <pgsql-hackers(at)lists(dot)postgresql(dot)org>
Subject: Re: zheap: a new storage format for PostgreSQL
Date: 2018-11-10 15:21:06
Message-ID: AM4PR0901MB1346FB51AD1CA33515A06985D2C70@AM4PR0901MB1346.eurprd09.prod.outlook.com
Views: Raw Message | Whole Thread | Download mbox
Thread:
Lists: pgsql-hackers

>>Thanks. Initializing the variable seems like the right fix here.

... just had a warning when recompiling from the latest sources on CentOS 7:

labels -Wmissing-format-attribute -Wformat-security -fno-strict-aliasing -fwrapv -fexcess-precision=standard -O2 -I../../../../src/include -D_GNU_SOURCE -I/usr/include/libxml2 -c -o tpd.o tpd.c
tpd.c: In function ‘TPDFreePage’:
tpd.c:1003:15: warning: variable ‘curblkno’ set but not used [-Wunused-but-set-variable]
BlockNumber curblkno = InvalidBlockNumber;
^

Not sure if this is important but as I could not find anything on this thread related to this I thought I'd report it

Regards
Daniel

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Peter Eisentraut 2018-11-10 15:23:38 Re: unused/redundant foreign key code
Previous Message David Rowley 2018-11-10 11:59:50 Re: speeding up planning with partitions