Re: Improve LATERAL join case in test memoize.sql

From: David Rowley <dgrowleyml(at)gmail(dot)com>
To: Richard Guo <guofenglinux(at)gmail(dot)com>
Cc: PostgreSQL-development <pgsql-hackers(at)postgresql(dot)org>
Subject: Re: Improve LATERAL join case in test memoize.sql
Date: 2023-01-23 23:31:59
Message-ID: CAApHDvqBSg01JixLAye58SH07JcMt2k0+7ed1u6Bgsh9PjC3ZA@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

On Mon, 16 Jan 2023 at 22:27, Richard Guo <guofenglinux(at)gmail(dot)com> wrote:
> I happened to notice we have the case in memoize.sql that tests for
> memoize node with LATERAL joins, which is
>
> -- Try with LATERAL joins
> SELECT explain_memoize('
> SELECT COUNT(*),AVG(t2.unique1) FROM tenk1 t1,
> LATERAL (SELECT t2.unique1 FROM tenk1 t2 WHERE t1.twenty = t2.unique1) t2
> WHERE t1.unique1 < 1000;', false);
>
> ISTM this is not the right query for the test.

> Attach a trivial patch for the change.

Good catch. I've applied this back to v14.

David

In response to

Browse pgsql-hackers by date

  From Date Subject
Next Message Euler Taveira 2023-01-23 23:32:08 Re: Time delayed LR (WAS Re: logical replication restrictions)
Previous Message Justin Pryzby 2023-01-23 23:28:14 Re: Cygwin cleanup