On Wed, Jun 19, 2024 at 6:02 PM Pavel Borisov <pashkin.e...@gmail.com> wrote: > On Wed, 19 Jun 2024 at 05:27, Alexander Korotkov <aekorot...@gmail.com> wrote: >> >> On Tue, Jun 18, 2024 at 4:14 PM Pavel Borisov <pashkin.e...@gmail.com> wrote: >> >> Controls if the query planner will produce a plan which will provide >> >> <literal>GROUP BY</literal> keys presorted in the order of keys of a >> >> child node of the plan, such as an index scan. When disabled, the query >> >> planner will produce a plan with <literal>GROUP BY</literal> keys only >> >> reordered to match >> >> the <literal>ORDER BY</literal> clause, if any. When enabled, the planner >> >> will try to produce a more efficient plan. The default value is on. >> > A correction of myself: presorted -> sorted, reordered ->sorted >> >> Thank you for your review. I think all of this make sense. Please, >> check the revised patch attached. > > To me patch v3 looks good.
Ok, thank you. I'm going to push this if no objections. ------ Regards, Alexander Korotkov Supabase
