"Joel Jacobson" <joel@compiler.org> writes:
> On Sun, Mar 1, 2026, at 22:12, Tom Lane wrote:
>> Aside: you could argue that failing to consider stanullfrac is wrong,
>> and maybe it is. But the more I looked at this code the more
>> convinced I got that it was only partially accounting for nulls
>> anyway. That seems like perhaps something to look into later.
> How about adjusting estfract for the null fraction before clamping?
This reminds me of the unfinished business at [1]. We really ought
to make it true that nulls never get into the hash table before
we assume that's so in costing. One of the things I was thinking
was being overlooked is the possibility of lots of nulls bloating
whichever hash bucket they get put in --- but if they aren't put
into a bucket then it's not wrong to ignore them here.
(Strictly speaking, that's still not so with non-strict hash operators,
but those are so rare that I don't mind not accounting for them.)
regards, tom lane
[1] https://www.postgresql.org/message-id/flat/3061845.1746486714@sss.pgh.pa.us
