Re: Use merge-based matching for MCVs in eqjoinsel

From: Ilia Evdokimov <ilya(dot)evdokimov(at)tantorlabs(dot)com>
To: David Geier <geidav(dot)pg(at)gmail(dot)com>, pgsql-hackers(at)lists(dot)postgresql(dot)org
Subject: Re: Use merge-based matching for MCVs in eqjoinsel
Date: 2025-09-10 13:56:03
Message-ID: 63b35674-6706-42db-bb8d-92ce03226d12@tantorlabs.com
Views: Whole Thread | Raw Message | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

Hi!

On 09.09.2025 12:22, David Geier wrote:
> Could you please do that with the latest attached patch so that we test
> it once more?

LGTM. Yes, I'll test this patch.

>
> Could you also run once more the JOB benchmark to get some test coverage
> on the SEMI join code (assuming it also uses SEMI joins)?

Unfortunately, the JOB benchmark does not contain semi join nodes.
However, TPC-DS does. I'll look for the queries with slowest planner
times there and check them.

I'll need some time to check both join and semi join cases with small
and large default_statistics_target. I'll share the results later.

>
> Once we've concluded on above and there are no objections, I will
> register this patch in the commit fest.

Sure. No problem.

--
Best regards,
Ilia Evdokimov,
Tantor Labs LLC,
https://tantorlabs.com

In response to

Browse pgsql-hackers by date

  From Date Subject
Next Message Robert Haas 2025-09-10 14:11:54 Re: plan shape work
Previous Message Natalya Aksman 2025-09-10 13:52:33 Re: Adding skip scan (including MDAM style range skip scan) to nbtree