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

From: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
To: "Haroon (dot)" <contact(dot)mharoon(at)gmail(dot)com>
Cc: Craig Ringer <craig(at)2ndquadrant(dot)com>, Michael Paquier <michael(dot)paquier(at)gmail(dot)com>, Haroon <muhammad(dot)haroon(at)2ndquadrant(dot)com>, Umair Shahid <umair(dot)shahid(at)gmail(dot)com>, PostgreSQL Hackers <pgsql-hackers(at)postgresql(dot)org>
Subject: Re: initdb issue on 64-bit Windows - (Was: [pgsql-packagers] PG 9.6beta2 tarballs are ready)
Date: 2016-06-29 22:56:30
Message-ID: 17585.1467240990@sss.pgh.pa.us
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

"Haroon ." <contact(dot)mharoon(at)gmail(dot)com> writes:
> On Sat, Jun 25, 2016 at 6:40 PM, Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)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

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Alvaro Herrera 2016-06-29 23:15:04 Re: initdb issue on 64-bit Windows - (Was: [pgsql-packagers] PG 9.6beta2 tarballs are ready)
Previous Message Tom Lane 2016-06-29 22:54:37 Re: primary_conninfo missing from pg_stat_wal_receiver