On 5/10/2022 02:45, Zhihong Yu wrote:
> Hi,
> For contain_placeholders():
>
> + if (IsA(node, Query))
> + return query_tree_walker((Query *) node, contain_placeholders,
> context, 0);
> + else if (IsA(node, PlaceHolderVar))
Fixed
>
> The `else` is not needed.
>
> For correlated_t struct, it would be better if the fields have comments.
Ok, I've added some comments.
>
> + * (for grouping, as an example). So, revert its
> status to
> + * a full valued entry.
>
> full valued -> fully valued
Fixed
--
regards,
Andrey Lepikhov
Postgres Professional
