jmazanec15 commented on code in PR #12050: URL: https://github.com/apache/lucene/pull/12050#discussion_r1091332482
########## lucene/core/src/java/org/apache/lucene/util/hnsw/HnswGraph.java: ########## @@ -182,10 +201,43 @@ public int nextInt() { public boolean hasNext() { return cur < size; } + } - /** The number of elements in this iterator * */ - public int size() { - return size; + /** Nodes iterator based on set representation of nodes. */ + public static class SetNodesIterator extends NodesIterator { Review Comment: Actually, it is required to have a "size()" method, so a better name might be CollectionNodesIterator. -- 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