msokolov commented on PR #11860: URL: https://github.com/apache/lucene/pull/11860#issuecomment-1320161291
+1 to the awesomeness - thanks for iterating on this fruit! - how high-hanging it is depends on one's perspective I guess. I have to say I am mildly amused that we are now using IndexedDISI to access the node neighbors / "postings" since the very initial hacky implementation of this hnsw index was based on SortedNumericDocValues, which is very similar under the hood I think. We decided to implement a new codec instead so that this could evolve independently, which I think made sense at the time, but it seems we've finally come full circle, at least sharing the implementation if not the actual field format. -- 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
