Thread

Re: Use merge-based matching for MCVs in eqjoinsel

Ilia Evdokimov <ilya.evdokimov@tantorlabs.com> — 2025-09-10T13:56:03Z

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