ok, sure, go ahead ;) Tommaso
2015-05-23 0:41 GMT+02:00 Edward J. Yoon <[email protected]>: > 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 >
