On Tue, Aug 30, 2022 at 10:04 AM Tom Lane <tgl@sss.pgh.pa.us> wrote:
Richard Guo <guofenglinux@gmail.com> writes: > To be concrete, I mean something like this:
> + /* Update the cost and width fields if we have added any > new columns. */ > + if (!equal(epq_path->pathtarget->exprs, target->exprs)) > + set_pathtarget_cost_width(root, target);
Wouldn't a list_length() comparison be sufficient? equal() seems like overkill.
Yeah, list_length comparison would do, as we only append new columns here. Thanks for the suggestion!