On Wed, Apr 12, 2023 at 5:04 PM David Rowley <dgrowle...@gmail.com> wrote:

>
> With the attached patch, that turns into:
>

The concept of startup_tuples for a WindowAgg looks good to me, but I
can't follow up with the below line:

+ return clamp_row_est(partition_tuples * DEFAULT_INEQ_SEL);

# select count(*) over() from tenk1 limit 1;
 count
-------
 10000  -->  We need to scan all the tuples.

Should we just return clamp_row_est(partition_tuples)?


-- 
Best Regards
Andy Fan

Reply via email to