From: Antonin Houska <ah@cybertec.at>
To: Ashutosh Bapat <ashutosh.bapat@enterprisedb.com>
cc: pgsql-hackers@postgresql.org
Subject: Re: advanced partition matching algorithm for partition-wise join
In-reply-to: 
 <CAFjFpRfdXpuSu0pxON3dKcr8WndJkaXLzHUVax_Laod0Tgc6UQ@mail.gmail.com>
References: 
 <CAFjFpRdjQvaUEV5DJX3TW6pU5eq54NCkadtxHX2JiJG_GvbrCA@mail.gmail.com>
 <CAKcux6nqCb7+zJBEF=QQzzHOB9acf2rMU+L609eWnkLtkKb5Vw@mail.gmail.com>
 <CAFjFpReNYx3814Twfg5rMMwCaOo8fzSmyhNLT8QmcFOnwiTJBQ@mail.gmail.com>
 <CAKcux6kinAwQFbk9Ce5_04UdKhrRm+SvpM-Cwz_Zc2Yqja+CYQ@mail.gmail.com>
 <CAFjFpRfdXpuSu0pxON3dKcr8WndJkaXLzHUVax_Laod0Tgc6UQ@mail.gmail.com>
Comments: In-reply-to Ashutosh Bapat <ashutosh.bapat@enterprisedb.com>
   message dated "Tue, 05 Sep 2017 16:34:18 +0530."
MIME-Version: 1.0
Content-Type: text/plain; charset=utf-8
Content-Transfer-Encoding: quoted-printable
Date: Thu, 07 Sep 2017 16:04:14 +0200
Message-ID: <2103.1504793054@localhost>
Precedence: bulk
Sender: pgsql-hackers-owner@postgresql.org

Ashutosh Bapat <ashutosh.bapat@enterprisedb.com> wrote:

> I have fixed the issues which were marked as TODOs in the attached
> patches. Also, I have included your test change patch in my series of
> patches.

I've noticed that partition_bounds_merge() is called twice from
make_join_rel():

 * build_join_rel -> build_joinrel_partition_info -> partition_bounds_merge

 * try_partition_wise_join -> partition_bounds_merge

Is this intentional, or just a thinko?

--=20
Antonin Houska
Cybertec Sch=C3=B6nig & Sch=C3=B6nig GmbH
Gr=C3=B6hrm=C3=BChlgasse 26
A-2700 Wiener Neustadt
Web: http://www.postgresql-support.de, http://www.cybertec.at


--=20
Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-hackers