Re: set_rel_pathlist function unnecessary params.

2024-09-30 Thread Tom Lane
Kirill Reshke writes: > I happened to notice that `set_rel_pathlist` params, RelOptInfo *rel > and RangeTblEntry *rte are > unnecessary, because upon all usages, > `rte=root->simple_rte_array[rti]` and > `rel=root->simple_rel_array[rti]` holds. What's the point of providing > the same information

set_rel_pathlist function unnecessary params.

2024-09-30 Thread Kirill Reshke
I happened to notice that `set_rel_pathlist` params, RelOptInfo *rel and RangeTblEntry *rte are unnecessary, because upon all usages, `rte=root->simple_rte_array[rti]` and `rel=root->simple_rel_array[rti]` holds. What's the point of providing the same information 3 times? Is it kept like that for e