Tembrel commented on this pull request.
> + * @return a new instance of SimpleTimeLimiter that uses executorService
+ */
+ public static SimpleTimeLimiter createSimpleTimeLimiter(ExecutorService
executorService) {
+ try {
+ if (CREATE_STL != null) {
+ return (SimpleTimeLimiter) CREATE_STL.invoke(null,
executorService);
+ } else if (CONSTRUCT_STL != null) {
+ return CONSTRUCT_STL.newInstance(executorService);
+ }
+ // fall through
+ } catch (IllegalAccessException iae) {
+ // fall through
+ } catch (InstantiationException ie) {
+ // fall through
+ } catch (InvocationTargetException ite) {
+ throw Throwables.propagate(ite.getCause());
I think the right thing to throw in the static initialization block, instead of
`Throwables.propagate` is `NoSuchMethodError`, instead of wrapping
`NoSuchMethodException` with an unchecked exception.
--
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds/pull/1133#discussion_r135400692
