[ONAMI-88] Consistent policy for exception handling

• Type: Improvement
• Status: Closed
• Priority: Minor
• Resolution: Fixed
• Affects Version/s: configuration-6.3.0
• Fix Version/s: configuration-6.3.0
• Component/s: configuration
• Assignee: Mikhail Mazursky
• Reporter: Mikhail Mazursky
• Created: 25/Feb/13 14:25
• Updated: 22/Feb/14 13:09
• Resolved: 01/Apr/13 11:01

Description

No consistent policy for exception handling - for example SQLTimestampConverter catches Throwable and rethrows it as ProvisionException but CharsetConverter do not catch anything at all. Should we catch at all or Guice handles this itself?

I guess we should rethrow only checked exceptions. Also no need to catch Errors in such places in any case (catch Exception vs Throwable).