Closed aubi closed 2 years ago
The previous implementation threw InvocationTargetException in contextual call. The new implementation throws the original exception, which makes it easier to understand the chain of causes of the exceptions.
In TCK, this passes the assert in ManagedScheduledExecutorDefinitionServlet.testCompletedFutureMSE: https://github.com/jakartaee/concurrency/blob/master/tck/src/main/java/ee/jakarta/tck/concurrent/spec/ManagedScheduledExecutorService/resourcedef/ManagedScheduledExecutorDefinitionServlet.java#L229
The previous implementation threw InvocationTargetException in contextual call. The new implementation throws the original exception, which makes it easier to understand the chain of causes of the exceptions.
In TCK, this passes the assert in ManagedScheduledExecutorDefinitionServlet.testCompletedFutureMSE: https://github.com/jakartaee/concurrency/blob/master/tck/src/main/java/ee/jakarta/tck/concurrent/spec/ManagedScheduledExecutorService/resourcedef/ManagedScheduledExecutorDefinitionServlet.java#L229