I think it would make sense to allow the validate method to throw Exception.
On Mon, Mar 20, 2017, 11:21 PM Jean-Baptiste Onofré <[email protected]> wrote: > Hi Ted, > > validate() is supposed to throw runtime exception (IllegalStateException, > RuntimeException, ...) to "traverse" the executor. > > Regards > JB > > On 03/21/2017 01:56 AM, Ted Yu wrote: > > Hi, > > I was reading HDFSFileSource.java where: > > > > @Override > > public void validate() { > > ... > > } catch (IOException | InterruptedException e) { > > throw new RuntimeException(e); > > } > > > > Why is validate() not declared to throw any exception ? > > If validation doesn't pass, there is nothing to clean up ? > > > > Thanks > > > > -- > Jean-Baptiste Onofré > [email protected] > http://blog.nanthrax.net > Talend - http://www.talend.com >
