mikemccand commented on issue #12744: URL: https://github.com/apache/lucene/issues/12744#issuecomment-1793487558
> Does that mean every values, including the ones with low-address will use the same bpv as the high-address nodes? PagedGrowableWriter already enlarge the bpv [automatically](https://github.com/apache/lucene/blob/8fa0de2743e87dd264619632978c8dc68323947a/lucene/core/src/java/org/apache/lucene/util/packed/GrowableWriter.java#L86), so maybe we could always set the initial bpv to a small value (like 8?) OK so I think normally `PagedGrowableWriter` would indeed be more compact if earlier values used smaller bpv than later values, *and* the paging was small enough so that there were enough pages to have separate mutables with different `bpv`? But in our usage here as a hash table, the inputs will more or less randomly scatter across the pages to the point where all pages will soon have to update to the large (current) bpv on insert anyways? -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: issues-unsubscr...@lucene.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: issues-unsubscr...@lucene.apache.org For additional commands, e-mail: issues-h...@lucene.apache.org
