guojialiang92 commented on PR #14417:
URL: https://github.com/apache/lucene/pull/14417#issuecomment-2766081605

   Thanks, @vigyasharma 
   > I think we can add a couple more tests to make it robust.
   > 
   > 1. Some tests around concurrency –  index with multiple threads, then 
advance the counter in one of the threads, and validate behavior. You can look 
at `ThreadedIndexingAndSearchingTestCase` and its derived tests for motivation.
   > 2. A test for the crash-recovery scenario, which I suppose it the primary 
use case. We could make the writer index a bunch of docs, then kill it, start a 
new writer on the same index, and advance its counter.
   
   I have added the following tests according to the suggestions: 
   1. For crash-recovery scenario, I have added 
`TestIndexWriter#testAdvanceSegmentCounterInCrashAndRecoveryScenario`.
   2. For scenarios with multi-threaded concurrent writes, I have modified the 
test `ThreadedIndexingAndSearchingTestCase`, in the concurrent write thread, 
increased the logic of randomly modifying the segment counter, and checked 
after all write threads ended.


-- 
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

Reply via email to