It looks like we have to throw the RuntimeException with proper message for ClassNotFoundException and IOException in GraphJobRunner. I can take this task.
On Sat, May 23, 2015 at 6:53 AM, Edward J. Yoon <[email protected]> wrote: > +1 > > > On Friday, May 22, 2015, Tommaso Teofili <[email protected]> wrote: >> >> Hi all, >> >> as a semi minor issue I've just noticed, we have several places where we >> have code blocks like: >> >> try { >> ... >> } catch (Exception e) { >> e.printStacktrace(); >> } >> >> I'd really like to get rid of such blocks as that's probably a sign of >> missing proper error handling, even logging errors may be ok, but at least >> it'd require us to more carefully think to what to do in case of errors. >> >> What do you think? >> >> Regards, >> Tommaso > > > > -- > Best Regards, Edward J. Yoon -- Best Regards, Edward J. Yoon
