Re: Question about some changes in 11.3

2019-06-12 Thread Tom Lane
Mat Arye writes: > On Mon, Jun 3, 2019 at 4:07 PM Tom Lane wrote: >> Hm. I'd say this was already broken by the invention of >> apply_scanjoin_target_to_paths; perhaps 11-before-11.3 managed to >> still work for you, but it's not hard to envision applications of >> set_rel_pathlist_hook for

Re: Question about some changes in 11.3

2019-06-05 Thread Mat Arye
Thanks for taking a look at this Tom. On Mon, Jun 3, 2019 at 4:07 PM Tom Lane wrote: > Mat Arye writes: > > On Fri, May 24, 2019 at 5:05 PM Mat Arye wrote: > >> 11.3 included some change to partition table planning. Namely > >> commit 925f46f ("Fix handling of targetlist SRFs when scan/join >

Re: Question about some changes in 11.3

2019-06-03 Thread Tom Lane
Mat Arye writes: > On Fri, May 24, 2019 at 5:05 PM Mat Arye wrote: >> 11.3 included some change to partition table planning. Namely >> commit 925f46f ("Fix handling of targetlist SRFs when scan/join relation is >> known empty.") seems to redo all paths for partitioned tables >> in

Re: Question about some changes in 11.3

2019-05-28 Thread Amit Langote
Hi Mat, On 2019/05/25 6:05, Mat Arye wrote: > Hi, > > 11.3 included some change to partition table planning. Namely > commit 925f46f ("Fix handling of targetlist SRFs when scan/join relation is > known empty.") seems to redo all paths for partitioned tables > in apply_scanjoin_target_to_paths.

Re: Question about some changes in 11.3

2019-05-28 Thread Mat Arye
On Fri, May 24, 2019 at 5:05 PM Mat Arye wrote: > Hi, > > 11.3 included some change to partition table planning. Namely > commit 925f46f ("Fix handling of targetlist SRFs when scan/join relation is > known empty.") seems to redo all paths for partitioned tables > in

Question about some changes in 11.3

2019-05-24 Thread Mat Arye
Hi, 11.3 included some change to partition table planning. Namely commit 925f46f ("Fix handling of targetlist SRFs when scan/join relation is known empty.") seems to redo all paths for partitioned tables in apply_scanjoin_target_to_paths. It clears the paths in: ``` if (rel_is_partitioned)