paulirwin commented on issue #1090:
URL: https://github.com/apache/lucenenet/issues/1090#issuecomment-2574343701

   I can't easily reproduce this one with that seed, which points to a 
concurrency bug (or at least one that is not reproducible due to concurrency), 
and that is indicated by the use of ThreadJobs in the test. But I think your 
theory is sound here, it lines up with the message and stack trace (assuming 
aggressive inlining likely from PGO), and the solution makes sense to me. I 
also reviewed ByteBufferIndexInput just to be on the safe side, and since 
`Remaining` would be 0 for an empty byte buffer, it should not cause any 
problems (and the code above it returns that anyways if length is 0). I'll get 
a PR for this one.


-- 
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: dev-unsubscr...@lucenenet.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Reply via email to