Home | About | Sematext search-lucene.com search-hadoop.com
 Search Hadoop and all its subprojects:

Switch to Threaded View
Accumulo >> mail # dev >> Review Request 23887: ACCUMULO-3012 - DefaultConfiguration


Copy link to this message
-
Re: Review Request 23887: ACCUMULO-3012 - DefaultConfiguration

This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/23887/#review48757
Mostly looks good. Small exception rename suggestion.
core/src/main/java/org/apache/accumulo/core/conf/ConfigSanityCheck.java
<https://reviews.apache.org/r/23887/#comment85503>

    Can we call this SanityCheckException?
    
    (CheckException is too generic, and intuitively seems to have something to do with "checked exceptions", which isn't the case.)
- Christopher Tubbs
On July 25, 2014, 1:08 p.m., Bill Havanki wrote: