qxggg opened a new issue, #15698: URL: https://github.com/apache/dubbo/issues/15698
### Pre-check - [x] I am sure that all the content I provide is in English. ### Search before asking - [x] I had searched in the [issues](https://github.com/apache/dubbo/issues?q=is%3Aissue) and found no similar feature requirement. ### Apache Dubbo Component Java SDK (apache/dubbo) ### Descriptions see #15671 and #15695 ```java @Override public void run(Timeout timeout) { DefaultFuture future = DefaultFuture.getFuture(requestID); if (future == null || future.isDone()) { return; } ExecutorService executor = future.getExecutor(); if (executor != null && !executor.isShutdown()) { try { executor.execute(() -> notifyTimeout(future)); } catch (RejectedExecutionException e) { notifyTimeout(future); throw e; } } else { notifyTimeout(future); } } ``` We have already solved #15671, but in PR #15695, @RainYuY thinks that we need another PR to change it so that it does not call notifyTimeout, because it’s not actually a timeout.(When error was caught). Do you think we should do that? If so, I can work on it. ### Related issues #15671 #15695 ### Are you willing to submit a pull request to fix on your own? - [x] Yes I am willing to submit a pull request on my own! ### Code of Conduct - [x] I agree to follow this project's [Code of Conduct](https://www.apache.org/foundation/policies/conduct) -- 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] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
