On Thu, Aug 27, 2026 at 8:36 PM David Rowley <[email protected]> wrote:
> On Thu, 27 Aug 2026 at 19:36, Richard Guo <[email protected]> wrote:
> > For the same reason as those commits, I'd also leave this master-only.
> > It is a v17 regression, but planner fixes risk destabilizing plans in
> > stable branches, and this doesn't seem severe enough to be an
> > exception.

> IMO, disabling a node type is quite a clear request that you only want
> that node type used if there's no alternative. I don't think the
> normal reasons for not wanting to destabilise plans here apply to
> this.

Makes sense.  I don't think anyone would get upset that we stopped
picking a plan containing a node they had disabled.

Maybe 47c110f77 and 6e466e1e8 should have been back-patched too?

Anyway, let's back-patch this one at least.

- Richard


Reply via email to