grouplens / grapht

Grapht dependency injector.
Other
25 stars 9 forks source link

Revisit checked-ness of exceptions #117

Closed mdekstrand closed 7 years ago

mdekstrand commented 9 years ago

As per discussion on the LensKit mailing list, we would like to consider moving Grapht to using unchecked exceptions and less exception wrapping. This will make it easier to debug components that use Grapht, by passing through the underlying exceptions more directly.

We can make LensKit use unchecked exceptions even without making the corresponding change to Grapht. This might make sense due to the fact that Grapht will be getting most (interesting) exceptions as InvocationTargetExceptions anwyay. However, we should re-evaluate Grapht's exceptions prior to LensKit 3.0.

mdekstrand commented 7 years ago

We will keep the Grapht exceptions as-is.