Dear Önder, Thanks for updating the patch! I will check it later. Currently I just reply to your comments.
> Hmm, I couldn't realize this comment earlier. So you suggest "slow" here > refers to the additional function call "GetRelationIdentityOrPK"? If so, yes > I'll update that. Yes I meant to say that, because functions will be called like: GetRelationIdentityOrPK() -> RelationGetPrimaryKeyIndex() -> RelationGetIndexList() -> .. and according to comments last one seems to do the heavy lifting. > Makes sense, simplified the function. Though, it is always hard to pick good > names for these kinds of helper functions. I picked > GenerateDummySelectPlannerInfoForRelation(), does that sound good to you as > well? I could not find any better naming than yours. Best Regards, Hayato Kuroda FUJITSU LIMITED
