(servicecomb-java-chassis) branch 2.8.x updated: [#4516]guard wait at least 30 seconds (#4525)

[liubao]

This is an automated email from the ASF dual-hosted git repository.

liubao pushed a commit to branch 2.8.x
in repository https://gitbox.apache.org/repos/asf/servicecomb-java-chassis.git


The following commit(s) were added to refs/heads/2.8.x by this push:
     new bd2d65cf2 [#4516]guard wait at least 30 seconds (#4525)
bd2d65cf2 is described below

commit bd2d65cf246278065ac8f07c46b831cd9b54947e
Author: liubao68 <bi...@qq.com>
AuthorDate: Sat Sep 14 18:03:43 2024 +0800

    [#4516]guard wait at least 30 seconds (#4525)
---
 .../servicecomb/core/provider/consumer/SyncResponseExecutor.java     | 5 ++---
 1 file changed, 2 insertions(+), 3 deletions(-)

diff --git 
a/core/src/main/java/org/apache/servicecomb/core/provider/consumer/SyncResponseExecutor.java
 
b/core/src/main/java/org/apache/servicecomb/core/provider/consumer/SyncResponseExecutor.java
index cf65384d3..228f21f53 100644
--- 
a/core/src/main/java/org/apache/servicecomb/core/provider/consumer/SyncResponseExecutor.java
+++ 
b/core/src/main/java/org/apache/servicecomb/core/provider/consumer/SyncResponseExecutor.java
@@ -98,8 +98,7 @@ public class SyncResponseExecutor implements Executor {
 
     // In invocation handlers, may call other microservices, invocation
     // timeout may be much longer than request timeout.
-    // But this is quite rare, for simplicity, default two times of request 
timeout.
-    // If users need longer timeout, can configure invocation timeout.
-    return invocation.getOperationMeta().getConfig().getMsRequestTimeout() * 2;
+    // For simplicity, default 30000 or two times of request timeout.
+    return 
Math.max(invocation.getOperationMeta().getConfig().getMsRequestTimeout() * 2, 
30000);
   }
 }