Ordering.natural is used 69 times in the Druid codebase, vs 5 for Comparator.naturalOrder. I think @AlexanderSaydakov could be forgiven for thinking Ordering.natural is the way to go.
Please, keep review comments to substantive design, coding, or style issues, rather than personal preference. If you want to start a discussion creating new style rules for Druid as a whole, a mailing list discussion or github issue is a better forum for doing that. [ Full content available at: https://github.com/apache/incubator-druid/pull/5712 ] This message was relayed via gitbox.apache.org for [email protected]
