[I] segmentInfos.replace() doesn't set userData [lucene]

[via GitHub]

Shibi-bala opened a new issue, #12637:
URL: https://github.com/apache/lucene/issues/12637

   ### Description
   
   Found that the [replace 
method](https://github.com/qcri/solr-6/blob/master/lucene/core/src/java/org/apache/lucene/index/SegmentInfos.java#L875-L878)
 doesn't set `userData` with the new user data from `other`. Unsure if this is 
an oversight, but if it is, I have a PR up [here
   ](https://github.com/apache/lucene/pull/12626).
   
   Existing:
   ```
     void replace(SegmentInfos other) {
       rollbackSegmentInfos(other.asList());
       lastGeneration = other.lastGeneration;
     }
   ```
   
   Fix:
   ```
     void replace(SegmentInfos other) {
       rollbackSegmentInfos(other.asList());
       lastGeneration = other.lastGeneration;
       userData = other.userData;
     }
   ```
   
   ### Version and environment details
   
   Lucene version 9.7


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