On 6/17/26 01:51, Enrique Sánchez wrote:
I think it's useful to know the estimated number of groups to
understand why the planner chooses IncrementalSort. The patch looks clean.
There are no tests for this change, although it should be safe, you
might want to consider adding one to
`src/test/regress/sql/incremental_sort.sql`.
Memoize has similar planner estimation output in EXPLAIN, but has no
regression tests for it either. Besides, in `inremental_sort` test all
EXPLAIN calls use COSTS OFF, so the new line would not appear there at
all. To be honest, I don't see any reason to display it in regression tests.
--
Best regards,
Ilia Evdokimov,
Tantor Labs LLC,
https://tantorlabs.com/
