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

Поиск
Список
Период
Сортировка
От Peter Eisentraut
Тема Re: Bug: pg_get_viewdef() fails on GRAPH_TABLE views with lateral column references
Дата
Msg-id f2977de9-73fb-4ced-a759-b53cb42c98be@eisentraut.org
обсуждение
Ответ на Re: Bug: pg_get_viewdef() fails on GRAPH_TABLE views with lateral column references  (SATYANARAYANA NARLAPURAM <satyanarlapuram@gmail.com>)
Список 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




В списке pgsql-hackers по дате отправления: