Re: Remove dead macro exec_subplan_get_plan

2023-04-04 Thread Gregory Stark (as CFM)
On Fri, 16 Sept 2022 at 03:33, Zhang Mingli wrote: > > On Sep 16, 2022, 14:47 +0800, Richard Guo , wrote: > > How about add it to the CF to not lose track of it? > > Will add it, thanks~ I guess not losing track of it is only helpful if we do eventually commit it. Otherwise we would rather lose t

Re: Remove dead macro exec_subplan_get_plan

2022-09-16 Thread Zhang Mingli
On Sep 16, 2022, 14:47 +0800, Richard Guo , wrote: > > On Tue, Sep 6, 2022 at 12:39 AM Zhang Mingli wrote: > > Macro exec_subplan_get_plan is not used anymore. > > Attach a patch to remove it. > > How about add it to the CF to not lose track of it? Will add it, thanks~ Regards, Zhang Mingli

Re: Remove dead macro exec_subplan_get_plan

2022-09-15 Thread Richard Guo
On Tue, Sep 6, 2022 at 12:39 AM Zhang Mingli wrote: > Macro exec_subplan_get_plan is not used anymore. > Attach a patch to remove it. > How about add it to the CF to not lose track of it? Thanks Richard

Re: Remove dead macro exec_subplan_get_plan

2022-09-06 Thread Zhang Mingli
Hi,all Regards, Zhang Mingli On Sep 6, 2022, 10:22 +0800, Richard Guo , wrote: > > On Tue, Sep 6, 2022 at 1:18 AM Tom Lane wrote: > > Zhang Mingli writes: > > > Macro exec_subplan_get_plan is not used anymore. > > > Attach a patch to remove it. > > > > Hm, I wonder why it's not used anymore.  Ma

Re: Remove dead macro exec_subplan_get_plan

2022-09-05 Thread Richard Guo
On Tue, Sep 6, 2022 at 1:18 AM Tom Lane wrote: > Zhang Mingli writes: > > Macro exec_subplan_get_plan is not used anymore. > > Attach a patch to remove it. > > Hm, I wonder why it's not used anymore. Maybe we no longer need > that list at all? If we do, should use of the macro be > re-introduc

Re: Remove dead macro exec_subplan_get_plan

2022-09-05 Thread Tom Lane
Zhang Mingli writes: > Macro exec_subplan_get_plan is not used anymore. > Attach a patch to remove it. Hm, I wonder why it's not used anymore. Maybe we no longer need that list at all? If we do, should use of the macro be re-introduced in the accessors? regards, tom lan

Remove dead macro exec_subplan_get_plan

2022-09-05 Thread Zhang Mingli
Hi, Macro exec_subplan_get_plan is not used anymore. Attach a patch to remove it. Regards, Zhang Mingli vn-0001-exec_subplan_get_plan-is-not-used.patch Description: Binary data