elharo commented on issue #1993: URL: https://github.com/apache/maven-resolver/issues/1993#issuecomment-5064189455
In SmartExecutor.Limited.submit(Runnable), catching an InterruptedException during semaphore.acquire() restores the thread's interrupt flag but silently drops the task without executing or enqueueing it. Because submit(Runnable) does not return a future, the caller relies entirely on the method's normal return as confirmation that the task was accepted. By returning normally without handling the task, the method creates a state of silent non-execution. In a dependency resolution tool, this can cause a build to incorrectly report success despite failing to resolve the complete artifact graph. This should be fixed. The method must ensure the task either executes or explicitly signals failure to the caller, typically by throwing an unchecked exception (such as a RejectedExecutionException or a wrapped RuntimeException) when the thread is interrupted before the task can be submitted. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
