Re: [HACKERS] Compilation broken when OPTIMIZER_DEBUG is set

2016-02-29 Thread Tom Lane
Michael Paquier  writes:
> On Mon, Feb 29, 2016 at 8:44 PM, salvador fandino  wrote:
>> allpaths.c: In function ‘debug_print_rel’:
>> allpaths.c:2943:50: error: ‘RelOptInfo {aka struct RelOptInfo}’ has no
>> member named ‘width’

> Indeed. The width is now part of rel->reltarget.width. See for example
> the attached while I bumped on this email..

Pushed, thanks.

regards, tom lane


-- 
Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-hackers


Re: [HACKERS] Compilation broken when OPTIMIZER_DEBUG is set

2016-02-29 Thread Michael Paquier
On Mon, Feb 29, 2016 at 8:44 PM, salvador fandino  wrote:
> gcc -Wall -Wmissing-prototypes -Wpointer-arith -Wdeclaration-after-statement
> -Wendif-labels -Wmissing-format-attribute -Wformat-security
> -fno-strict-aliasing -fwrapv -fexcess-precision=standard -g -O0
> -DOPTIMIZER_DEBUG -I../../../../src/include -D_GNU_SOURCE   -c -o allpaths.o
> allpaths.c
> allpaths.c: In function ‘debug_print_rel’:
> allpaths.c:2943:50: error: ‘RelOptInfo {aka struct RelOptInfo}’ has no
> member named ‘width’
>   printf("): rows=%.0f width=%d\n", rel->rows, rel->width);
>   ^
>
> git bisect points to commit 19a541143a09c067ec8cac77ec6a64eb5b1b662b "Add an
> explicit representation of the output targetlist to Paths."

Indeed. The width is now part of rel->reltarget.width. See for example
the attached while I bumped on this email..
-- 
Michael


optimizer-debug-fix.patch
Description: binary/octet-stream

-- 
Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-hackers