jpountz commented on a change in pull request #426:
URL: https://github.com/apache/lucene/pull/426#discussion_r743484713



##########
File path: lucene/core/src/java/org/apache/lucene/util/InPlaceMergeSorter.java
##########
@@ -18,7 +18,10 @@
 
 /**
  * {@link Sorter} implementation based on the merge-sort algorithm that merges 
in place (no extra
- * memory will be allocated). Small arrays are sorted with insertion sort.
+ * memory will be allocated). Small arrays are sorted with binary sort.
+ *
+ * <p>This algorithm is stable. It's especially suited to sorting small lists 
where we'd rather
+ * optimize for avoiding allocating memory for this task.

Review comment:
       While we're adding this, maybe also mention that it performs well on 
lists that are already sorted.




-- 
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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]



---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to