I'm only talking about line 170. You pass the exception to log.error(); you shouldn't need to call e.printStackTrace().