@gianm I suspect that most of that 69 usages of `Ordering.natural()` were introduced when Druid had to support Java 7, or after that, simply because `Comparator.naturalOrder()` is lesser known. `Comparator.naturalOrder()` is slightly preferred because it's standard, but the difference is so minor that it's OK to ignore such comments for PR authors. I leave them because PR authors might be unaware of this API, but when they become aware they prefer to use `Comparator.naturalOrder()` themselves.
[ Full content available at: https://github.com/apache/incubator-druid/pull/5712 ] This message was relayed via gitbox.apache.org for [email protected]
