> On Sep 28, 2025, at 06:23, Tom Lane <[email protected]> wrote: > > + /* Iterate over targets, adding acceptable ones to the result > list */ > + foreach_ptr(TargetEntry, tle, *targetlist) > + { > + /* Ignore junk TLEs */ > + if (tle->resjunk) > + continue;
Do we want to specifically check “ctid”? If a user does: ``` select ctid, col1, col2, ... from t group by all; ``` It would be equivalent to no group by. Combing “select ctid” with “group by all” seems totally useless, but when users do such things, we can optimize that. Best regards, -- Chao Li (Evan) HighGo Software Co., Ltd. https://www.highgo.com/
