jmazanec15 commented on issue #12570: URL: https://github.com/apache/lucene/issues/12570#issuecomment-1726544154
For dynamically changing M, I would be a little hesitant IMO. It could lead to inconsistent behavior, where some segments are searched very fast and others slow. If we were to support it, I would say that the initialization from graph should not be used because this might hurt the integrity of the graph. > I don't know for sure if we really restrict the number of connections when adding back-connections :/ Don't we limit back connections here: https://github.com/apache/lucene/blob/main/lucene/core/src/java/org/apache/lucene/util/hnsw/HnswGraphBuilder.java#L276-L279? -- 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
