You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@kyuubi.apache.org by fe...@apache.org on 2022/06/25 15:05:06 UTC
[incubator-kyuubi] branch master updated: [KYUUBI #886][FOLLOWUP] Support to reload hadoop conf for KyuubiTHttpFrontendService
This is an automated email from the ASF dual-hosted git repository.
feiwang 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 9bc0aa676 [KYUUBI #886][FOLLOWUP] Support to reload hadoop conf for KyuubiTHttpFrontendService
9bc0aa676 is described below
commit 9bc0aa67644b75802764588b3c666ea4969eaa16
Author: Fei Wang <fw...@ebay.com>
AuthorDate: Sat Jun 25 23:04:58 2022 +0800
[KYUUBI #886][FOLLOWUP] Support to reload hadoop conf for KyuubiTHttpFrontendService
### _Why are the changes needed?_
Support to reload hadoop conf for KyuubiTHttpFrontendService.
See details in #2742
### _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
- [x] [Run test](https://kyuubi.apache.org/docs/latest/develop_tools/testing.html#running-tests) locally before make a pull request
Closes #2899 from turboFei/http_followup.
Closes #886
c480aa20 [Fei Wang] [KYUUBI #886][FOLLOWUP] Support to reload hadoop conf for KyuubiTHttpFrontendService
Authored-by: Fei Wang <fw...@ebay.com>
Signed-off-by: Fei Wang <fw...@ebay.com>
---
.../scala/org/apache/kyuubi/server/KyuubiTHttpFrontendService.scala | 3 +++
1 file changed, 3 insertions(+)
diff --git a/kyuubi-server/src/main/scala/org/apache/kyuubi/server/KyuubiTHttpFrontendService.scala b/kyuubi-server/src/main/scala/org/apache/kyuubi/server/KyuubiTHttpFrontendService.scala
index 565589010..b6beb2cbc 100644
--- a/kyuubi-server/src/main/scala/org/apache/kyuubi/server/KyuubiTHttpFrontendService.scala
+++ b/kyuubi-server/src/main/scala/org/apache/kyuubi/server/KyuubiTHttpFrontendService.scala
@@ -23,6 +23,7 @@ import javax.security.sasl.AuthenticationException
import javax.servlet.{ServletContextEvent, ServletContextListener}
import org.apache.commons.lang3.SystemUtils
+import org.apache.hadoop.conf.Configuration
import org.apache.hive.service.rpc.thrift.{TCLIService, TOpenSessionReq}
import org.apache.thrift.protocol.TBinaryProtocol
import org.eclipse.jetty.http.HttpMethod
@@ -62,6 +63,8 @@ final class KyuubiTHttpFrontendService(
private var server: Option[Server] = None
private val APPLICATION_THRIFT = "application/x-thrift"
+ override protected def hadoopConf: Configuration = KyuubiServer.getHadoopConf()
+
/**
* Configure Jetty to serve http requests. Example of a client connection URL:
* http://localhost:10000/servlets/thrifths2/ A gateway may cause actual target