Re: Between Node

From: Rod Taylor <rbt(at)zort(dot)ca>
To: Christopher Kings-Lynne <chriskl(at)familyhealth(dot)com(dot)au>
Cc: Bruce Momjian <pgman(at)candle(dot)pha(dot)pa(dot)us>, Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>, pgsql-patches(at)postgresql(dot)org
Subject: Re: Between Node
Date: 2002-07-18 11:36:11
Message-ID: 1026992174.280.58.camel@jester
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-patches

On Thu, 2002-07-18 at 00:56, Christopher Kings-Lynne wrote:
> I believe Tom mentioned that having BETWEEN as a node would allow him to do
> some sort of funky optimiser improvement. Is that already done or does he
> still need to do it?

I don't know what improvement this would be. I only touched the parser
and executor.

It only executes each segment of the between tree once so the only
optimization I can think of would be to try to execute the most likely
to fail leaf first.

> > -----Original Message-----
> > From: pgsql-patches-owner(at)postgresql(dot)org
> > [mailto:pgsql-patches-owner(at)postgresql(dot)org]On Behalf Of Bruce Momjian
> > Sent: Thursday, 18 July 2002 12:42 PM
> > To: Rod Taylor
> > Cc: pgsql-patches(at)postgresql(dot)org
> > Subject: Re: [PATCHES] Between Node
> >
> >
> >
> > Patch applied. Thanks.
> >
> > ------------------------------------------------------------------
> > ---------
> >
> >
> > Rod Taylor wrote:
> > > Finished the Between patch Christopher started.
> > >
> > > Implements between (symmetric / asymmetric) as a node.
> > >
> > > Executes the left or right expression once, makes a Const out of the
> > > resulting Datum and executes the >=, <= portions out of the Const sets.
> > >
> > > Of course, the parser does a fair amount of preparatory work for this to
> > > happen.
> >
> > [ Attachment, skipping... ]
> >
> > >
> > > ---------------------------(end of broadcast)---------------------------
> > > TIP 6: Have you searched our list archives?
> > >
> > > http://archives.postgresql.org
> >
> > --
> > Bruce Momjian | http://candle.pha.pa.us
> > pgman(at)candle(dot)pha(dot)pa(dot)us | (610) 853-3000
> > + If your life is a hard drive, | 830 Blythe Avenue
> > + Christ can be your backup. | Drexel Hill, Pennsylvania 19026
> >
> > ---------------------------(end of broadcast)---------------------------
> > TIP 4: Don't 'kill -9' the postmaster
> >
>
>
> ---------------------------(end of broadcast)---------------------------
> TIP 2: you can get off all lists at once with the unregister command
> (send "unregister YourEmailAddressHere" to majordomo(at)postgresql(dot)org)
>

In response to

Responses

Browse pgsql-patches by date

  From Date Subject
Next Message Tom Lane 2002-07-18 13:31:32 Re: CLUSTER patch
Previous Message Magnus Naeslund(f) 2002-07-18 09:41:51 [PATCH] Win32 native fixes after SSL updates (+more)