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