Re: initdb issue on 64-bit Windows - (Was: [pgsql-packagers] PG 9.6beta2 tarballs are ready) - Mailing list pgsql-hackers

From Tom Lane
Subject Re: initdb issue on 64-bit Windows - (Was: [pgsql-packagers] PG 9.6beta2 tarballs are ready)
Date
Msg-id 17585.1467240990@sss.pgh.pa.us
Whole thread Raw
In response to Re: initdb issue on 64-bit Windows - (Was: [pgsql-packagers] PG 9.6beta2 tarballs are ready)  ("Haroon ." <contact.mharoon@gmail.com>)
Responses Re: initdb issue on 64-bit Windows - (Was: [pgsql-packagers] PG 9.6beta2 tarballs are ready)  (Alvaro Herrera <alvherre@2ndquadrant.com>)
List pgsql-hackers
"Haroon ." <contact.mharoon@gmail.com> writes:
> On Sat, Jun 25, 2016 at 6:40 PM, Tom Lane <tgl@sss.pgh.pa.us> wrote:
>> This leads to a couple of suggestions for dodging the problem:
>> 
>> 2. Don't pass the original formal parameter to
>> get_foreign_key_join_selectivity, ie do something like
>> 
>> static double
>> calc_joinrel_size_estimate(PlannerInfo *root,
>> RelOptInfo *outer_rel,
>> RelOptInfo *inner_rel,
>> double outer_rows,
>> double inner_rows,
>> SpecialJoinInfo *sjinfo,
>> -                                                  List *restrictlist)
>> +                                                  List *orig_restrictlist)
>> {
>> JoinType        jointype = sjinfo->jointype;
>> +       List       *restrictlist = orig_restrictlist;
>> Selectivity fkselec;
>> Selectivity jselec;
>> Selectivity pselec;
>> 
>> 
> The problem appears to be related to 'taking the address of a formal
> parameter'. NOT passing the original formal parameter to
> get_foreign_key_join_selectivity fixes it (dodges the problem) on VS2013.

Thanks for investigating!  I'll go commit that change.  I wish someone
would put up a buildfarm critter using VS2013, though.
        regards, tom lane



pgsql-hackers by date:

Previous
From: Tom Lane
Date:
Subject: Re: primary_conninfo missing from pg_stat_wal_receiver
Next
From: Alvaro Herrera
Date:
Subject: Re: initdb issue on 64-bit Windows - (Was: [pgsql-packagers] PG 9.6beta2 tarballs are ready)