Hi Mr.Pyhalov.
Alexander Pyhalov <a.pyhalov@postgrespro.ru>
Thursday, September 28, 2023 3:16 PM
> It seems to me that *fdw postfixes don't clarify things, but just make naming more ugly.
I have removed *fdw postfixes.
> + * Adding these Vars and PlaceHolderVars to PathTarget,
> + * FDW cannot deparse this by the original List of SortGroupClauses.
> + * So, before this adding process,
> + * setGroupClausePartial generates another Pathtarget and another
> + * List of SortGroupClauses for FDW.
>
> It seems that something like:
>
> /*
> * Modified PathTarget cannot be used by FDW as-is to deparse this statement.
> * So, before modifying PathTarget, setGroupClausePartial generates
> * another Pathtarget and another list List of SortGroupClauses
> * to make deparsing possible.
> */
>
> sounds better.
Thank you for the suggested modifications. I have modified it according to your suggestion.
Sincerely yours,
Yuuki Fujii
--
Yuuki Fujii
Information Technology R&D Center Mitsubishi Electric Corporation