Re: Unwanted expression simplification in PG12b2

From: Robert Haas <robertmhaas(at)gmail(dot)com>
To: Darafei Komяpa Praliaskouski <me(at)komzpa(dot)net>
Cc: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>, PostgreSQL Developers <pgsql-hackers(at)lists(dot)postgresql(dot)org>
Subject: Re: Unwanted expression simplification in PG12b2
Date: 2019-09-23 12:58:00
Message-ID: CA+TgmoYrwqrMjMWf8CFNQZoLvd0+EmR5ZYba775H4yPOJz+xFw@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

On Sun, Sep 22, 2019 at 7:47 AM Darafei "Komяpa" Praliaskouski
<me(at)komzpa(dot)net> wrote:
> A heuristic I believe should help my case (and I hardly imagine how it can break others) is that in presence of Gather, all the function calls that are parallel safe should be pushed into it.

The cost of pushing data through the Sort is not necessarily
insignificant. Your functions are (IIUC) extremely expensive, so it's
worth going to any length to reduce the time spent evaluating them.
However, if someone has ||(text,text) in the tlist, that might be the
wrong approach, because it's not saving much to compute that earlier
and it might make the sort a lot wider, especially if de-TOASTing is
involved.

--
Robert Haas
EnterpriseDB: http://www.enterprisedb.com
The Enterprise PostgreSQL Company

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message John Bester 2019-09-23 13:43:34 Proposal: Better query optimization for "NOT IN" clause
Previous Message Andrew Gierth 2019-09-23 12:16:36 Re: Efficient output for integer types