You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@doris.apache.org by GitBox <gi...@apache.org> on 2022/05/25 12:01:10 UTC

[GitHub] [incubator-doris] pengxiangyu opened a new pull request, #9779: Add backend_rpc_timeout_second in AdminConfig

pengxiangyu opened a new pull request, #9779:
URL: https://github.com/apache/incubator-doris/pull/9779

   # Proposed changes
   
   Issue Number: close #xxx
   
   ## Problem Summary:
   
   Add backend_rpc_timeout_second in AdminConfig
   
   ## Checklist(Required)
   
   1. Does it affect the original behavior: (No)
   2. Has unit tests been added: (No)
   3. Has document been added or modified: (No)
   4. Does it need to update dependencies: (No)
   5. Are there any changes that cannot be rolled back: (No)
   
   ## Further comments
   


-- 
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: commits-unsubscribe@doris.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For additional commands, e-mail: commits-help@doris.apache.org


[GitHub] [incubator-doris] morningman merged pull request #9779: Add backend_rpc_timeout_second in AdminConfig

Posted by GitBox <gi...@apache.org>.
morningman merged PR #9779:
URL: https://github.com/apache/incubator-doris/pull/9779


-- 
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: commits-unsubscribe@doris.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For additional commands, e-mail: commits-help@doris.apache.org


[GitHub] [incubator-doris] github-actions[bot] commented on pull request #9779: Add backend_rpc_timeout_second in AdminConfig

Posted by GitBox <gi...@apache.org>.
github-actions[bot] commented on PR #9779:
URL: https://github.com/apache/incubator-doris/pull/9779#issuecomment-1138658032

   PR approved by anyone and no changes requested.


-- 
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: commits-unsubscribe@doris.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For additional commands, e-mail: commits-help@doris.apache.org


[GitHub] [incubator-doris] morningman commented on a diff in pull request #9779: Add backend_rpc_timeout_second in AdminConfig

Posted by GitBox <gi...@apache.org>.
morningman commented on code in PR #9779:
URL: https://github.com/apache/incubator-doris/pull/9779#discussion_r881765585


##########
fe/fe-core/src/main/java/org/apache/doris/common/Config.java:
##########
@@ -1651,4 +1651,8 @@ public class Config extends ConfigBase {
 
     @ConfField
     public static boolean enable_vectorized_load = false;
+
+    @ConfField(mutable = false, masterOnly = true)
+    public static int backend_rpc_timeout_ms = 60000; // 1 min

Review Comment:
   Please add comment for this config and also modify the config document
   `admin-manual/config/fe-config.md`



-- 
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: commits-unsubscribe@doris.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For additional commands, e-mail: commits-help@doris.apache.org


[GitHub] [incubator-doris] github-actions[bot] commented on pull request #9779: Add backend_rpc_timeout_second in AdminConfig

Posted by GitBox <gi...@apache.org>.
github-actions[bot] commented on PR #9779:
URL: https://github.com/apache/incubator-doris/pull/9779#issuecomment-1138658005

   PR approved by at least one committer and no changes requested.


-- 
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: commits-unsubscribe@doris.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For additional commands, e-mail: commits-help@doris.apache.org