pseudo-nymous opened a new pull request, #13421: URL: https://github.com/apache/lucene/pull/13421
### Description closes #13372 TestOrdinalMap#testRamBytesUsed was failing for the case when there are multiple Default instances for PackedInts.NullReader. Currently, we return 0 when the instance is built with default size i.e. 256. Ref: org.apache.lucene.util.packed.PackedInts.NullReader#ramBytesUsed But this was not accounted for in the Accumulator being used. Made changes to skip the object if the reader is of default static instance. Passes failing UT. `./gradlew :lucene:core:test --tests "org.apache.lucene.index.TestOrdinalMap.testRamBytesUsed" -Ptests.seed=55680AD930225FC5 -Ptests.nightly=true` -- 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
