Bools are frustrating, because they don't carry semantic intent. Java 8 supports switch statements on String now, so you could just use a switch statement here, and that makes the patch significantly smaller.
Or you could use an enum instead of Strings. Either way, the patch becomes much smaller if you can keep the `CryptoTest.CRYPTO_ON_CONF` and `CryptoTest.CRYPTO_OFF_CONF` constants throughout. [ Full content available at: https://github.com/apache/accumulo/pull/641 ] This message was relayed via gitbox.apache.org for [email protected]
