On Tue Oct 7, 2025 at 6:17 PM -03, Tom Lane wrote: > "Matheus Alcantara" <[email protected]> writes: >> On Tue Oct 7, 2025 at 1:51 PM -03, Tom Lane wrote: >>> Also, I don't think it's the job of this patch to provide test >>> coverage for dshash. That should be quite well covered already. > >> When I was mentioning to test that we can grow the dshash correctly it's >> because the v9 patch has a logic to grow the array stored on dshash >> entry value that currently is not being cover by the tests. I'm not >> saying to test the dshash internal logic which I agree that it's not the >> job of this patch. Sorry for being confusing. > > Ah, yeah, I misunderstood what you meant. I agree that covering that > "Grow the array" stanza is a good idea, in fact I said the same thing > a little bit ago. > Yeah, I just saw your response after I sent the email, which I agree with all the points. So I think that we are on the same "page" now.
-- Matheus Alcantara
