benwtrent commented on issue #15051:
URL: https://github.com/apache/lucene/issues/15051#issuecomment-3184339830

   I think the failure is this line:
   ```
               MergeThread newMergeThread = super.getMergeThread(mergeSource, 
merge);
               mergeThreadSet.add(newMergeThread);
   ```
   
   `getMergeThread` now adds the merge thread set directly. 
   
   See: 
https://github.com/apache/lucene/pull/15015/files#diff-ceb8ec6fe5807682cfb691a8ec52bcc672fb7c5eeb6922c80da4c075f7f003c8
   
   If we think this is an OK change, I am cool with it. However, I would much 
rather keep the CMS completely untouched as its very brittle.
   
   @vigyasharma @corecursion 


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