On Tue, 21 Mar 2023 at 09:41, David Rowley <dgrowle...@gmail.com> wrote: > Because that's being changed in v16, I think it might also be a good > idea to fix the hit_ratio calculation problem reported by David > Johnston in [1]. In the attached, I've adjusted David's calculation > slightly so that we divide by Max(ndistinct, est_cache_entries) > instead of ndistinct. This saves from overestimating when ndistinct > is smaller than est_cache_entries. I'd rather fix this now for v16 > than wait until v17 and further adjust the Memoize costing.
I've now pushed this change. David
