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

zabetak pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/hive.git


The following commit(s) were added to refs/heads/master by this push:
     new ae0cabffeaf HIVE-26631: Remove unused Thrift config parameters 
login.timeout and exponential.backoff.slot.length (xiuzhu9527 reviewed by 
Stamatis Zampetakis)
ae0cabffeaf is described below

commit ae0cabffeaf284a6d2ec13a6993c87770818fbb9
Author: xiuzhu9527 <1406823...@qq.com>
AuthorDate: Fri Oct 14 13:31:51 2022 +0800

    HIVE-26631: Remove unused Thrift config parameters login.timeout and 
exponential.backoff.slot.length (xiuzhu9527 reviewed by Stamatis Zampetakis)
    
    Closes #3672
---
 common/src/java/org/apache/hadoop/hive/conf/HiveConf.java          | 7 -------
 .../org/apache/hive/service/cli/thrift/ThriftBinaryCLIService.java | 4 ----
 2 files changed, 11 deletions(-)

diff --git a/common/src/java/org/apache/hadoop/hive/conf/HiveConf.java 
b/common/src/java/org/apache/hadoop/hive/conf/HiveConf.java
index 12688c3f0fe..27fdddc47c9 100644
--- a/common/src/java/org/apache/hadoop/hive/conf/HiveConf.java
+++ b/common/src/java/org/apache/hadoop/hive/conf/HiveConf.java
@@ -4092,13 +4092,6 @@ public class HiveConf extends Configuration {
         "Minimum number of Thrift worker threads"),
     
HIVE_SERVER2_THRIFT_MAX_WORKER_THREADS("hive.server2.thrift.max.worker.threads",
 500,
         "Maximum number of Thrift worker threads"),
-    HIVE_SERVER2_THRIFT_LOGIN_BEBACKOFF_SLOT_LENGTH(
-        "hive.server2.thrift.exponential.backoff.slot.length", "100ms",
-        new TimeValidator(TimeUnit.MILLISECONDS),
-        "Binary exponential backoff slot time for Thrift clients during login 
to HiveServer2,\n" +
-        "for retries until hitting Thrift client timeout"),
-    HIVE_SERVER2_THRIFT_LOGIN_TIMEOUT("hive.server2.thrift.login.timeout", 
"20s",
-        new TimeValidator(TimeUnit.SECONDS), "Timeout for Thrift clients 
during login to HiveServer2"),
     
HIVE_SERVER2_THRIFT_WORKER_KEEPALIVE_TIME("hive.server2.thrift.worker.keepalive.time",
 "60s",
         new TimeValidator(TimeUnit.SECONDS),
         "Keepalive time (in seconds) for an idle worker thread. When the 
number of workers exceeds min workers, " +
diff --git 
a/service/src/java/org/apache/hive/service/cli/thrift/ThriftBinaryCLIService.java
 
b/service/src/java/org/apache/hive/service/cli/thrift/ThriftBinaryCLIService.java
index ec339da985f..8fc728573db 100644
--- 
a/service/src/java/org/apache/hive/service/cli/thrift/ThriftBinaryCLIService.java
+++ 
b/service/src/java/org/apache/hive/service/cli/thrift/ThriftBinaryCLIService.java
@@ -100,10 +100,6 @@ public class ThriftBinaryCLIService extends 
ThriftCLIService {
 
       // Server args
       int maxMessageSize = 
hiveConf.getIntVar(HiveConf.ConfVars.HIVE_SERVER2_THRIFT_MAX_MESSAGE_SIZE);
-      int requestTimeout = (int) 
hiveConf.getTimeVar(HiveConf.ConfVars.HIVE_SERVER2_THRIFT_LOGIN_TIMEOUT,
-          TimeUnit.SECONDS);
-      int beBackoffSlotLength = (int) hiveConf
-          
.getTimeVar(HiveConf.ConfVars.HIVE_SERVER2_THRIFT_LOGIN_BEBACKOFF_SLOT_LENGTH, 
TimeUnit.MILLISECONDS);
       TThreadPoolServer.Args sargs = new 
TThreadPoolServer.Args(serverSocket).processorFactory(processorFactory)
           .transportFactory(transportFactory).protocolFactory(new 
TBinaryProtocol.Factory())
           .inputProtocolFactory(new TBinaryProtocol.Factory(true, true, 
maxMessageSize, maxMessageSize))

Reply via email to