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

ulyssesyou 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 683675b  [KYUUBI #1146] Move KinitAuxiliaryService to kyuubi-server 
module
683675b is described below

commit 683675be921570cf0eddbc18796716718da46530
Author: Brian Yue <[email protected]>
AuthorDate: Sun Sep 26 17:57:25 2021 +0800

    [KYUUBI #1146] Move KinitAuxiliaryService to kyuubi-server module
    
    <!--
    Thanks for sending a pull request!
    
    Here are some tips for you:
      1. If this is your first time, please read our contributor guidelines: 
https://kyuubi.readthedocs.io/en/latest/community/contributions.html
      2. If the PR is related to an issue in 
https://github.com/apache/incubator-kyuubi/issues, add '[KYUUBI #XXXX]' in your 
PR title, e.g., '[KYUUBI #XXXX] Your PR title ...'.
      3. If the PR is unfinished, add '[WIP]' in your PR title, e.g., 
'[WIP][KYUUBI #XXXX] Your PR title ...'.
    -->
    
    ### _Why are the changes needed?_
    <!--
    Please clarify why the changes are needed. For instance,
      1. If you add a feature, you can talk about the use case of it.
      2. If you fix a bug, you can clarify why it is a bug.
    -->
    Close #1146.
    
    ### _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.readthedocs.io/en/latest/develop_tools/testing.html#running-tests)
 locally before make a pull request
    
    Closes #1156 from byyue/feature-1146.
    
    Closes #1146
    
    6b677723 [Brian Yue] Move KinitAuxiliaryService to kyuubi-server module
    
    Authored-by: Brian Yue <[email protected]>
    Signed-off-by: ulysses-you <[email protected]>
---
 .../main/scala/org/apache/kyuubi/server}/KinitAuxiliaryService.scala   | 3 ++-
 .../src/main/scala/org/apache/kyuubi/server/KyuubiServer.scala         | 2 +-
 .../scala/org/apache/kyuubi/server}/KinitAuxiliaryServiceSuite.scala   | 3 ++-
 3 files changed, 5 insertions(+), 3 deletions(-)

diff --git 
a/kyuubi-common/src/main/scala/org/apache/kyuubi/service/KinitAuxiliaryService.scala
 
b/kyuubi-server/src/main/scala/org/apache/kyuubi/server/KinitAuxiliaryService.scala
similarity index 97%
rename from 
kyuubi-common/src/main/scala/org/apache/kyuubi/service/KinitAuxiliaryService.scala
rename to 
kyuubi-server/src/main/scala/org/apache/kyuubi/server/KinitAuxiliaryService.scala
index a44738d..dd6f8e1 100644
--- 
a/kyuubi-common/src/main/scala/org/apache/kyuubi/service/KinitAuxiliaryService.scala
+++ 
b/kyuubi-server/src/main/scala/org/apache/kyuubi/server/KinitAuxiliaryService.scala
@@ -15,13 +15,14 @@
  * limitations under the License.
  */
 
-package org.apache.kyuubi.service
+package org.apache.kyuubi.server
 
 import java.util.concurrent.TimeUnit
 
 import org.apache.hadoop.security.UserGroupInformation
 
 import org.apache.kyuubi.config.KyuubiConf
+import org.apache.kyuubi.service.AbstractService
 import org.apache.kyuubi.util.{KyuubiHadoopUtils, ThreadUtils}
 
 class KinitAuxiliaryService() extends AbstractService("KinitAuxiliaryService") 
{
diff --git 
a/kyuubi-server/src/main/scala/org/apache/kyuubi/server/KyuubiServer.scala 
b/kyuubi-server/src/main/scala/org/apache/kyuubi/server/KyuubiServer.scala
index 785c788..d610b5b 100644
--- a/kyuubi-server/src/main/scala/org/apache/kyuubi/server/KyuubiServer.scala
+++ b/kyuubi-server/src/main/scala/org/apache/kyuubi/server/KyuubiServer.scala
@@ -26,7 +26,7 @@ import org.apache.kyuubi.config.KyuubiConf
 import org.apache.kyuubi.ha.HighAvailabilityConf._
 import org.apache.kyuubi.ha.client.ServiceDiscovery
 import org.apache.kyuubi.metrics.{MetricsConf, MetricsSystem}
-import org.apache.kyuubi.service.{AbstractBackendService, 
AbstractFrontendService, KinitAuxiliaryService, Serverable}
+import org.apache.kyuubi.service.{AbstractBackendService, 
AbstractFrontendService, Serverable}
 import org.apache.kyuubi.util.{KyuubiHadoopUtils, SignalRegister}
 import org.apache.kyuubi.zookeeper.EmbeddedZookeeper
 
diff --git 
a/kyuubi-common/src/test/scala/org/apache/kyuubi/service/KinitAuxiliaryServiceSuite.scala
 
b/kyuubi-server/src/test/scala/org/apache/kyuubi/server/KinitAuxiliaryServiceSuite.scala
similarity index 96%
rename from 
kyuubi-common/src/test/scala/org/apache/kyuubi/service/KinitAuxiliaryServiceSuite.scala
rename to 
kyuubi-server/src/test/scala/org/apache/kyuubi/server/KinitAuxiliaryServiceSuite.scala
index 7cc4ded..96aa269 100644
--- 
a/kyuubi-common/src/test/scala/org/apache/kyuubi/service/KinitAuxiliaryServiceSuite.scala
+++ 
b/kyuubi-server/src/test/scala/org/apache/kyuubi/server/KinitAuxiliaryServiceSuite.scala
@@ -15,12 +15,13 @@
  * limitations under the License.
  */
 
-package org.apache.kyuubi.service
+package org.apache.kyuubi.server
 
 import org.apache.hadoop.security.UserGroupInformation
 
 import org.apache.kyuubi.KerberizedTestHelper
 import org.apache.kyuubi.config.KyuubiConf
+import org.apache.kyuubi.service.ServiceState
 
 class KinitAuxiliaryServiceSuite extends KerberizedTestHelper {
 

Reply via email to