msokolov commented on PR #12050:
URL: https://github.com/apache/lucene/pull/12050#issuecomment-1379112305

   > To support this functionality, a couple of changes to current graph 
construction process needed to be made. OnHeapHnswGraph had to support out of 
order insertion. This is because the mapped ordinals of the nodes in the graph 
used for initialization are not necessarily the first X ordinals in the new 
graph.
   
   I'm having trouble wrapping my head around this. When we start merging some 
field, each segment seg has a graph with ordinals in [0,seg.size]. Why can't we 
preserve the ordinals from the largest segment, and then let the others fall 
where they may?


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