Re: pgsql: Report progress of CREATE INDEX operations

From: Andres Freund <andres(at)anarazel(dot)de>
To: Alvaro Herrera <alvherre(at)2ndquadrant(dot)com>
Cc: pgsql-committers(at)lists(dot)postgresql(dot)org
Subject: Re: pgsql: Report progress of CREATE INDEX operations
Date: 2019-04-02 19:00:03
Message-ID: 20190402190003.esiyxbmnm63jlydd@alap3.anarazel.de
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-committers

Hi,

On 2019-04-02 15:41:46 -0300, Alvaro Herrera wrote:
> On 2019-Apr-02, Alvaro Herrera wrote:
>
> > Report progress of CREATE INDEX operations
>
> Argh, I pushed an older version of the commit.

Note that longfin complains about something different than the output
files:

https://buildfarm.postgresql.org/cgi-bin/show_log.pl?nm=longfin&dt=2019-04-02%2018%3A49%3A03

lmgr.c:882:14: error: variable 'count' is used uninitialized whenever '?:' condition is false [-Werror,-Wsometimes-uninitialized]
progress ? &count : NULL));
^~~~~~~~
lmgr.c:883:12: note: uninitialized use occurs here
total += count;
^~~~~
lmgr.c:882:14: note: remove the '?:' if its condition is always true
progress ? &count : NULL));
^~~~~~~~~~~ ~~~~~~~
lmgr.c:878:14: note: initialize the variable 'count' to silence this warning
int count;
^
= 0
1 error generated.
make[4]: *** [lmgr.o] Error 1
make[4]: *** Waiting for unfinished jobs....

Greetings,

Andres Freund

In response to

Responses

Browse pgsql-committers by date

  From Date Subject
Next Message Alvaro Herrera 2019-04-02 19:03:52 pgsql: Prevent use of uninitialized variable
Previous Message Alvaro Herrera 2019-04-02 18:46:22 pgsql: Update expected output for modified catalog definition