[COMMITTERS] pgsql: Fixes in bloom contrib module

2016-04-02 Thread Teodor Sigaev
Fixes in bloom contrib module Looking at result of buildfarm member jaguarundi it seems to me that BloomOptions isn't inited sometime, but I don't see yet how it's possible. Nevertheless, check of signature length's is missed, so, add a limit of it. Also add missed GenericXLogAbort() in case of al

[COMMITTERS] pgsql: Prevent mark as deleted and as 'has free space' page in bloom mo

2016-04-02 Thread Teodor Sigaev
Prevent mark as deleted and as 'has free space' page in bloom module Vacuum might put page into list of pages with some free space and mark as deleted at the same time. Branch -- master Details --- http://git.postgresql.org/pg/commitdiff/e9e441c9fac6cbc0510cded6abb9d0e6b646ecaf Modified

[COMMITTERS] pgsql: Fix typo in pg_regress.c

2016-04-02 Thread Stephen Frost
Fix typo in pg_regress.c s/afer/after Pointed out by Andreas 'ads' Scherbaum Branch -- master Details --- http://git.postgresql.org/pg/commitdiff/62b5cd234ba982f71f2501f405a26ed80c92a229 Modified Files -- src/test/regress/pg_regress.c | 2 +- 1 file changed, 1 insertion(+),

[COMMITTERS] pgsql: Fix condition in e9e441c9fac6cbc0510cded6abb9d0e6b646ecaf

2016-04-02 Thread Teodor Sigaev
Fix condition in e9e441c9fac6cbc0510cded6abb9d0e6b646ecaf Comment is right, but if - not. Branch -- master Details --- http://git.postgresql.org/pg/commitdiff/9c50372d20aabdb23963f8d28651728d546acefb Modified Files -- contrib/bloom/blvacuum.c | 2 +- 1 file changed, 1 inserti

Re: [COMMITTERS] pgsql: XLogReader general code cleanup

2016-04-02 Thread Peter Eisentraut
On 03/30/2016 05:57 PM, Alvaro Herrera wrote: XLogReader general code cleanup This is causing compiler warnings on 32-bit environments(?): xlogreader.c: In function ‘XLogReadRecord’: xlogreader.c:334:7: error: format ‘%lu’ expects argument of type ‘long unsigned int’, but argument 5 has type

[COMMITTERS] pgsql: Suppress compiler warning.

2016-04-02 Thread Tom Lane
Suppress compiler warning. Some buildfarm members are showing "comparison is always false due to limited range of data type" complaints on this test, so #ifdef it out on machines with 32-bit int. Branch -- master Details --- http://git.postgresql.org/pg/commitdiff/45aae8e78967b37f285e996

[COMMITTERS] pgsql: Make all the declarations of WaitEventSetWaitBlock be marked "in

2016-04-02 Thread Tom Lane
Make all the declarations of WaitEventSetWaitBlock be marked "inline". The inconsistency here triggered compiler warnings on some buildfarm members, and it's surely pretty pointless. Branch -- master Details --- http://git.postgresql.org/pg/commitdiff/a1953f3a60cc7d1b8516d0b2c7e82ae8e924

[COMMITTERS] pgsql: Add missing "static".

2016-04-02 Thread Tom Lane
Add missing "static". Per buildfarm member pademelon. Branch -- master Details --- http://git.postgresql.org/pg/commitdiff/5a5b917184b630529635db2e037d298ad90c355d Modified Files -- contrib/bloom/blutils.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) -- Sent via p

Re: [COMMITTERS] pgsql: XLogReader general code cleanup

2016-04-02 Thread Alvaro Herrera
Peter Eisentraut wrote: > On 03/30/2016 05:57 PM, Alvaro Herrera wrote: > >XLogReader general code cleanup > > This is causing compiler warnings on 32-bit environments(?): Will fix. -- Álvaro Herrerahttp://www.2ndQuadrant.com/ PostgreSQL Development, 24x7 Support, Remote DBA, Tr