Github user bzz commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/458#issuecomment-158736103
Looks great, but how complicated is it to reproduce this bug?
If there is a simple case, it might be worth adding a test too.
Also a bit of explanation in comments in case of empty catch block (i.e why
is it ok to ignore it here) is of a [good
style](https://google.github.io/styleguide/javaguide.html#s6.2-caught-exceptions)
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at [email protected] or file a JIRA ticket
with INFRA.
---
