On Fri, Dec 30, 2022 at 11:00 AM Richard Guo <guofengli...@gmail.com> wrote:
> On Fri, Dec 9, 2022 at 5:16 PM Richard Guo <guofengli...@gmail.com> wrote: > >> Actually we do have checked PHVs for lateral references, earlier in >> create_lateral_join_info. But that time we only marked lateral_relids >> and direct_lateral_relids, without remembering the lateral expressions. >> So I'm wondering whether we can fix that by fetching Vars (or PHVs) of >> lateral references within PlaceHolderVars and remembering them in the >> baserel's lateral_vars. >> >> Attach a draft patch to show my thoughts. >> > > Update the patch to fix test failures. > Rebase the patch on HEAD as cfbot reminds. Thanks Richard
v3-0001-Check-lateral-refs-within-PHVs-for-memoize-cache-keys.patch
Description: Binary data