On Mon, Oct 9, 2023 at 12:13 PM David Rowley <dgrowle...@gmail.com> wrote:

> I've now pushed the patch that trims off the Pathkeys for the ORDER BY
> / DISTINCT aggregates.


Thanks for pushing!


> Those results are a bit noisy.  Perhaps a few more runs might yield
> more consistency, but it seems that there's not too much overhead to
> it. If I take the minimum value out of the 3 runs from each, it comes
> to about 1.5% extra time spent in planning.  Perhaps that's OK.


I agree that the overhead is acceptable, especially it only happens in
USE_ASSERT_CHECKING builds.

Thanks
Richard

Reply via email to