msokolov opened a new pull request, #13654: URL: https://github.com/apache/lucene/pull/13654
another in an ongoing series of patches to address merge stability test failures. Here we tweak the back-compat Lucene 94/95 writers so they will call the new connectComponents() method (via `getCompletedGraph()/finish()) when flushing. It's annoying we couldn't simply have changed HnswGraphBuilder.getGraph() since these already called that. However getGraph() is also called *while* building (to check RAM usage IIRC) so that wouldn't work.- I checked previous versions and I think we should be good there as they would never call into this code path even when merging, but it's a little hard to trace by eye ... it's possible we could still see a test fail for some older version. -- 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
