msfroh commented on PR #15558:
URL: https://github.com/apache/lucene/pull/15558#issuecomment-3747078175
Wow... @sgup432, those are some nice numbers. Any luck on reproducing and/or
fixing the latest test failure?
```
ERROR: 1 test has failed:
-
org.apache.lucene.search.TestLRUQueryCache.testDocValuesUpdatesDontBreakCache
(:lucene:core)
Test output:
/home/runner/work/lucene/lucene/lucene/core/build/test-results/test/outputs/OUTPUT-org.apache.lucene.search.TestLRUQueryCache.txt
Reproduce with: gradlew :lucene:core:test --tests
"org.apache.lucene.search.TestLRUQueryCache.testDocValuesUpdatesDontBreakCache"
-Ptests.asserts=true -Ptests.file.encoding=ISO-8859-1 -Ptests.gui=true
-Ptests.jvmargs= -Ptests.jvms=4 -Ptests.seed=D906E38BEA18E902
-Ptests.vectorsize=256
```
I don't necessarily see how doc value updates would be affected by the
structural part of your change.
Maybe it's the async cleanup? That test does seem to rely on items getting
evicted from the cache because of the DV update.
--
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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
