Re: [PR] Account for 0 graph size when initializing HNSW graph [lucene]

2024-10-31 Thread via GitHub
mayya-sharipova commented on PR #13964: URL: https://github.com/apache/lucene/pull/13964#issuecomment-2450001895 @john-wagster Thanks for the review. I tried to write tests, but it needs a lot of setup and mocks, and I thought it does't worth. But I plan to write integration kind of t

Re: [PR] Account for 0 graph size when initializing HNSW graph [lucene]

2024-10-31 Thread via GitHub
mayya-sharipova merged PR #13964: URL: https://github.com/apache/lucene/pull/13964 -- 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...@lu

[PR] Account for 0 graph size when initializing HNSW graph [lucene]

2024-10-29 Thread via GitHub
mayya-sharipova opened a new pull request, #13964: URL: https://github.com/apache/lucene/pull/13964 When initializing a joint graph from one of the segments' graphs, we always assume that a segment's graph is present. But later we want to explore an option where some segments will not