[KYUUBI apache#816] [KYUUBI 813] Fix requestTimeoutUnit in FrontendService

wuchunfu · yaooqinn · commit 0a87a254f726 · 2021-07-16T12:45:57.000+08:00
Fix requestTimeoutUnit in FrontendService apache#813 ### _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.readthedocs.io/en/latest/tools/testing.html#running-tests) locally before make a pull request Closes apache#816 from wuchunfu/813. Closes apache#816 d15797c [wuchunfu] [KYUUBI 813] Fix requestTimeoutUnit in FrontendService Authored-by: wuchunfu <319355703@qq.com> Signed-off-by: Kent Yao <yao@apache.org>
diff --git a/kyuubi-common/src/main/scala/org/apache/kyuubi/service/FrontendService.scala b/kyuubi-common/src/main/scala/org/apache/kyuubi/service/FrontendService.scala
@@ -88,7 +88,7 @@ class FrontendService private (name: String, be: BackendService, oomHook: Runnab
         .protocolFactory(new TBinaryProtocol.Factory)
         .inputProtocolFactory(
           new TBinaryProtocol.Factory(true, true, maxMessageSize, maxMessageSize))
-        .requestTimeout(requestTimeout).requestTimeoutUnit(TimeUnit.SECONDS)
+        .requestTimeout(requestTimeout).requestTimeoutUnit(TimeUnit.MILLISECONDS)
         .beBackoffSlotLength(beBackoffSlotLength)
         .beBackoffSlotLengthUnit(TimeUnit.MILLISECONDS)
         .executorService(executor)
diff --git a/kyuubi-main/src/main/scala/org/apache/kyuubi/client/KyuubiSyncThriftClient.scala b/kyuubi-main/src/main/scala/org/apache/kyuubi/client/KyuubiSyncThriftClient.scala
@@ -0,0 +1,5 @@
+package org.apache.kyuubi.client
+
+class KyuubiSyncThriftClient {
+
+}

-Original file line number
+Diff line change
@@ @@ -0,0 +1,5 @@ @@
 +package org.apache.kyuubi.client
++
 +class KyuubiSyncThriftClient {
++
 +}