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/11/27 09:46:02 UTC

[GitHub] [doris] caiconghui commented on a diff in pull request #9172: [enhancement](k8s) Support fqdn mode for be in k8s enviroment

caiconghui commented on code in PR #9172:
URL: https://github.com/apache/doris/pull/9172#discussion_r1032899046


##########
fe/fe-core/src/main/java/org/apache/doris/system/SystemInfoService.java:
##########
@@ -901,7 +915,8 @@ public void clear() {
         this.idToReportVersionRef = null;
     }
 
-    public static Pair<String, Integer> validateHostAndPort(String hostPort) throws AnalysisException {
+    public static Triple<String, String, Integer> getIpHostAndPort(String hostPort, boolean strictCheck)

Review Comment:
   because when pod is deconstructed,the ip of pod may not be obtained,to prevent that we cannot drop backend



-- 
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