[jira] [Work logged] (HIVE-27172) Add the HMS client connection timeout config

[ASF GitHub Bot (Jira)]

     [ 
https://issues.apache.org/jira/browse/HIVE-27172?focusedWorklogId=858792&page=com.atlassian.jira.plugin.system.issuetabpanels:worklog-tabpanel#worklog-858792
 ]

ASF GitHub Bot logged work on HIVE-27172:
-----------------------------------------

                Author: ASF GitHub Bot
            Created on: 24/Apr/23 21:09
            Start Date: 24/Apr/23 21:09
    Worklog Time Spent: 10m 
      Work Description: deniskuzZ commented on code in PR #4150:
URL: https://github.com/apache/hive/pull/4150#discussion_r1175784708


##########
standalone-metastore/metastore-common/src/main/java/org/apache/hadoop/hive/metastore/conf/MetastoreConf.java:
##########
@@ -402,6 +402,8 @@ public enum ConfVars {
             "has an infinite lifetime."),
     CLIENT_SOCKET_TIMEOUT("metastore.client.socket.timeout", 
"hive.metastore.client.socket.timeout", 600,
             TimeUnit.SECONDS, "MetaStore Client socket timeout in seconds"),
+    CLIENT_CONNECTION_TIMEOUT("metastore.client.connection.timeout", 
"hive.metastore.client.connection.timeout", 10,

Review Comment:
   @wecharyu, won't be socketTimeout = connectTimeout if not explicitly set?
   {code}
   public TSocket(TConfiguration config, String host, int port, int timeout)  {
       this(config, host, port, timeout, timeout);
     }
   public TSocket(TConfiguration config, String host, int port, int 
socketTimeout, int connectTimeout)  {}
   {code}





Issue Time Tracking
-------------------

    Worklog Id:     (was: 858792)
    Time Spent: 1h 40m  (was: 1.5h)

> Add the HMS client connection timeout config
> --------------------------------------------
>
>                 Key: HIVE-27172
>                 URL: https://issues.apache.org/jira/browse/HIVE-27172
>             Project: Hive
>          Issue Type: Task
>          Components: Hive
>            Reporter: Wechar
>            Assignee: Wechar
>            Priority: Major
>              Labels: pull-request-available
>          Time Spent: 1h 40m
>  Remaining Estimate: 0h
>
> Currently {{HiveMetastoreClient}} use {{CLIENT_SOCKET_TIMEOUT}} as both 
> socket timeout and connection timeout, it's not convenient for users to set a 
> smaller connection timeout.



--
This message was sent by Atlassian Jira
(v8.20.10#820010)