Re: [HACKERS] Calculation of param_source_rels in add_paths_to_joinrel

2016-11-23 Thread Ashutosh Bapat
Thanks Tom. On Thu, Nov 24, 2016 at 2:58 AM, Tom Lane wrote: > Ashutosh Bapat writes: >> On Sat, Nov 5, 2016 at 2:16 AM, Tom Lane wrote: >>> Ashutosh Bapat writes: Also, the way this

Re: [HACKERS] Calculation of param_source_rels in add_paths_to_joinrel

2016-11-23 Thread Tom Lane
Ashutosh Bapat writes: > On Sat, Nov 5, 2016 at 2:16 AM, Tom Lane wrote: >> Ashutosh Bapat writes: >>> Also, the way this code has been written, the declaration of variable >>> sjinfo masks the earlier

Re: [HACKERS] Calculation of param_source_rels in add_paths_to_joinrel

2016-11-10 Thread Ashutosh Bapat
On Sat, Nov 5, 2016 at 2:16 AM, Tom Lane wrote: > Ashutosh Bapat writes: >> There's code in add_paths_to_joinrel() which computes the set of >> target relations that should overlap parameterization of any proposed >> join path. >> ... >> The

Re: [HACKERS] Calculation of param_source_rels in add_paths_to_joinrel

2016-11-04 Thread Tom Lane
Ashutosh Bapat writes: > There's code in add_paths_to_joinrel() which computes the set of > target relations that should overlap parameterization of any proposed > join path. > ... > The calculations that follow are based on joinrel->relids (baserels > covered by

[HACKERS] Calculation of param_source_rels in add_paths_to_joinrel

2016-09-20 Thread Ashutosh Bapat
Hi, There's code in add_paths_to_joinrel() which computes the set of target relations that should overlap parameterization of any proposed join path. 120 /* 121 * Decide whether it's sensible to generate parameterized paths for this 122 * joinrel, and if so, which relations such