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: [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]

Reply via email to