[incubator-kyuubi] branch master updated: [KYUUBI #2461] Use the original host argument

[ul...@apache.org]

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

ulyssesyou pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/incubator-kyuubi.git


The following commit(s) were added to refs/heads/master by this push:
     new dcea90b01 [KYUUBI #2461] Use the original host argument
dcea90b01 is described below

commit dcea90b01e5e07fe11007cbfffba08c03e2246c3
Author: wforget <64...@qq.com>
AuthorDate: Mon Apr 25 11:47:31 2022 +0800

    [KYUUBI #2461] Use the original host argument
    
    ### _Why are the changes needed?_
    
    close #2461
    
    ### _How was this patch tested?_
    - [ ] Add some test cases that check the changes thoroughly including negative and positive cases if possible
    
    - [ ] Add screenshots for manual tests if appropriate
    
    - [ ] [Run test](https://kyuubi.apache.org/docs/latest/develop_tools/testing.html#running-tests) locally before make a pull request
    
    Closes #2462 from wForget/KYUUBI-2461.
    
    Closes #2461
    
    1a0a60e9 [wforget] [KYUUBI-2461] Use the original host argument
    
    Authored-by: wforget <64...@qq.com>
    Signed-off-by: ulysses-you <ul...@apache.org>
---
 .../main/scala/org/apache/kyuubi/ctl/ServiceControlCliArguments.scala  | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/kyuubi-ctl/src/main/scala/org/apache/kyuubi/ctl/ServiceControlCliArguments.scala b/kyuubi-ctl/src/main/scala/org/apache/kyuubi/ctl/ServiceControlCliArguments.scala
index 61171bfb5..d60fc3587 100644
--- a/kyuubi-ctl/src/main/scala/org/apache/kyuubi/ctl/ServiceControlCliArguments.scala
+++ b/kyuubi-ctl/src/main/scala/org/apache/kyuubi/ctl/ServiceControlCliArguments.scala
@@ -249,7 +249,8 @@ class ServiceControlCliArguments(args: Seq[String], env: Map[String, String] = s
     }
 
     try {
-      cliArgs = cliArgs.copy(host = InetAddress.getByName(cliArgs.host).getCanonicalHostName)
+      InetAddress.getByName(cliArgs.host)
+      cliArgs = cliArgs.copy(host = cliArgs.host)
     } catch {
       case _: Exception =>
         fail(s"Unknown host: ${cliArgs.host}")