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
