-
Notifications
You must be signed in to change notification settings - Fork 964
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
TestPackedInts.testPackedLongValues failing #13249
Labels
Comments
Git bisect shows: c41eb22 @original-brownbear mind taking a look? |
@benwtrent sure thing taking a look! |
original-brownbear
added a commit
to original-brownbear/lucene
that referenced
this issue
Mar 29, 2024
Correct estimate for singleton to return `0` and use custom accumulator in tests to fix assertions. Tried not doing this in apache#13232 but turns out we need a little complexity here since the singleton is recognized by the `RamUsageTester` in so far that is only counted once if it shows up repeatedly in an array or so. fixes apache#13249
Here we go: #13250 turns out we need to be a little tricky here after all for singletons + ram estimation. |
benwtrent
pushed a commit
that referenced
this issue
Apr 1, 2024
…3250) Correct estimate for singleton to return `0` and use custom accumulator in tests to fix assertions. Tried not doing this in #13232 but turns out we need a little complexity here since the singleton is recognized by the `RamUsageTester` in so far that is only counted once if it shows up repeatedly in an array or so. fixes #13249
benwtrent
pushed a commit
that referenced
this issue
Apr 1, 2024
…3250) Correct estimate for singleton to return `0` and use custom accumulator in tests to fix assertions. Tried not doing this in #13232 but turns out we need a little complexity here since the singleton is recognized by the `RamUsageTester` in so far that is only counted once if it shows up repeatedly in an array or so. fixes #13249
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Description
TestPackedInts.testPackedLongValues fails repeatably, example trace:
Gradle command to reproduce
./gradlew :lucene:core:test --tests "org.apache.lucene.util.packed.TestPackedInts.testPackedLongValues" -Ptests.jvms=2 -Ptests.jvmargs= -Ptests.seed=E73C4D01E8CF65EC -Ptests.gui=true -Ptests.file.encoding=US-ASCII -Ptests.vectorsize=128
The text was updated successfully, but these errors were encountered: