public inbox for [email protected]  
help / color / mirror / Atom feed
From: Tom Lane <[email protected]>
To: Tomas Vondra <[email protected]>
Cc: Jim Finnerty <[email protected]>
Cc: [email protected]
Subject: Re: On disable_cost
Date: Sat, 02 Nov 2019 11:04:58 -0400
Message-ID: <[email protected]> (raw)
In-Reply-To: <20191101170406.w6ybnafyvpxv2ltg@development>
References: <CAO0i4_SSPV9TVxbbTRVLOnCyewopcc147fBZy=f2ABk15eHS+g@mail.gmail.com>
	<CA+hUKGJ0NTtLB5dv5gbuDoRirtamC--wWGNoAT4=fF2Ev1bRdA@mail.gmail.com>
	<[email protected]>
	<CA+Tgmob4+Eu5LN4j1XEMQDtCFFLKuUCEJebND3bWrQL0-N7VTA@mail.gmail.com>
	<[email protected]>
	<20191101170406.w6ybnafyvpxv2ltg@development>

Tomas Vondra <[email protected]> writes:
> On Fri, Nov 01, 2019 at 09:30:52AM -0700, Jim Finnerty wrote:
>> re: coping with adding disable_cost more than once
>> 
>> Another option would be to have a 2-part Cost structure.  If disable_cost is
>> ever added to the Cost, then you set a flag recording this.  If any plans
>> exist that have no disable_costs added to them, then the planner chooses the
>> minimum cost among those, otherwise you choose the minimum cost path.

> Yeah, I agree having is_disabled flag, and treat all paths with 'true'
> as more expensive than paths with 'false' (and when both paths have the
> same value then actually compare the cost) is probably the way forward.

It would have to be a count, not a boolean --- for example, you want to
prefer a path that uses one disabled SeqScan over a path that uses two.

I'm with Andres in being pretty worried about the extra burden imposed
on add_path comparisons.

			regards, tom lane





view thread (20+ messages)  latest in thread

reply

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Reply to all the recipients using the --to and --cc options:
  reply via email

  To: [email protected]
  Cc: [email protected], [email protected], [email protected]
  Subject: Re: On disable_cost
  In-Reply-To: <[email protected]>

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

This inbox is served by agora; see mirroring instructions
for how to clone and mirror all data and code used for this inbox