Hi, I am looking at Flume NG source and sink implementations. I have noticed a common pattern, one of a few specific examples is:
https://reviews.apache.org/r/3741 In the start method, if we get an IOException in the start method, we log the error. However, when the process method is called a NPE will be throw because Htable is still null. Is there a reason we don't wrap the error in a runtime exception and throw it up? Another example is: https://reviews.apache.org/r/3831/ I am just curious if there is a reason we do log the error as opposed to throwing it as a RuntimeException. Cheers Brock -- Apache MRUnit - Unit testing MapReduce - http://incubator.apache.org/mrunit/
