Re: Bug: pg_get_viewdef() fails on GRAPH_TABLE views with lateral column references

From: Peter Eisentraut <peter(at)eisentraut(dot)org>
To: SATYANARAYANA NARLAPURAM <satyanarlapuram(at)gmail(dot)com>, Ashutosh Bapat <ashutosh(dot)bapat(dot)oss(at)gmail(dot)com>
Cc: PostgreSQL Hackers <pgsql-hackers(at)lists(dot)postgresql(dot)org>
Subject: Re: Bug: pg_get_viewdef() fails on GRAPH_TABLE views with lateral column references
Date: 2026-04-24 07:18:12
Message-ID: f2977de9-73fb-4ced-a759-b53cb42c98be@eisentraut.org
Views: Whole Thread | Raw Message | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

On 21.04.26 10:02, SATYANARAYANA NARLAPURAM wrote:
> The code doesn't explain why it adds the dummy context but it seemed
> intentional. But it's not used at other places like deparsing WHERE
> clause in element patterns or that in the graph_table itself. Since a
> lateral reference is allowed in COLUMNS clause as well, it doesn't
> make sense not to pass a context with lateral namespaces. Also there
> is no comment explaining the dummy context. So your fix looks good to
> me. I adjusted the surrounding code a bit.
>
> I adjusted an existing view for the testing instead of adding a new
> one with all the additional objects. Since that view definition was
> getting more complex, I formatted the DDL to be more readable.
>
> I also think that we should use prettyFlags to deparse all GRAPH_TABLE
> components in a human readable form. But that's out of the scope for
> this patch.
>
> PFA updated patch.
>
> Thank you for updating the patch. It applies cleanly and the related
> tests are passing.

committed

In response to

Browse pgsql-hackers by date

  From Date Subject
Next Message lakshmi 2026-04-24 07:18:41 Re: ECPG: inconsistent behavior with the document in “GET/SET DESCRIPTOR.”
Previous Message shveta malik 2026-04-24 07:17:03 Re: Fix race condition in pg_get_publication_tables with concurrent DROP TABLE