The semantics of this is kind of ugly (i.e. non-intuitive). We're testing that a method that returns a Boolean to return null. Yes, I realize the declared return type (`Boolean`) could be null, but that confuses the intent of a method named `isXxxxx()`
The logic in `public boolean isConcurrencyChecksEnabled()` is more intuitive. [ Full content available at: https://github.com/apache/geode/pull/3156 ] This message was relayed via gitbox.apache.org for [email protected]
