Skip to content
GitLab
  • Menu
Projects Groups Snippets
  • Help
    • Help
    • Support
    • Community forum
    • Submit feedback
  • Sign in
  • I Incubator Kyuubi
  • Project information
    • Project information
    • Activity
    • Labels
    • Members
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributors
    • Graph
    • Compare
    • Locked Files
  • Issues 0
    • Issues 0
    • List
    • Boards
    • Service Desk
    • Milestones
    • Requirements
  • Merge requests 0
    • Merge requests 0
  • CI/CD
    • CI/CD
    • Pipelines
    • Jobs
    • Schedules
    • Test Cases
  • Deployments
    • Deployments
    • Environments
    • Releases
  • Packages & Registries
    • Packages & Registries
    • Package Registry
    • Infrastructure Registry
  • Monitor
    • Monitor
    • Incidents
  • Analytics
    • Analytics
    • Value stream
    • CI/CD
    • Code review
    • Insights
    • Issue
    • Repository
  • Wiki
    • Wiki
  • Snippets
    • Snippets
  • Activity
  • Graph
  • Create a new issue
  • Jobs
  • Commits
  • Issue Boards
Collapse sidebar
  • 小 白蛋
  • Incubator Kyuubi
  • Repository
Switch branch/tag
  • incubator-kyuubi
  • externals
  • kyuubi-spark-sql-engine
  • pom.xml
Find file BlameHistoryPermalink
  • ulysses-you's avatar
    [KYUUBI #386] Clean up discovery service before engine stop in connection share level · aa878e4d
    ulysses-you authored Mar 04, 2021
    ![ulysses-you](https://badgen.net/badge/Hello/ulysses-you/green) [![Closes #387](https://badgen.net/badge/Preview/Closes%20%23387/blue)](https://github.com/yaooqinn/kyuubi/pull/387) ![330](https://badgen.net/badge/%2B/330/red) ![52](https://badgen.net/badge/-/52/green) ![18](https://badgen.net/badge/commits/18/yellow) ![Test Plan](https://badgen.net/badge/Missing/Test%20Plan/ff0000) [&#10088;?&#10089;](https://pullrequestbadge.com/?utm_medium=github&utm_source=yaooqinn&utm_campaign=badge_info)<!-- PR-BADGE: PLEASE DO NOT REMOVE THIS COMMENT -->
    
    <!--
    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/yaooqinn/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.
    -->
    In connection level, we will release engine if the session connection is closed but we don't clean up the discovery service (i.e. the namespace in zookeeper). That may cause the disk stress after running a long time.
    
    ### _How was this patch tested?_
    Add some new suites in spark engine moudle with zk emmbedded.
    
    Closes #387 from ulysses-you/issue-386.
    
    ccb1112 [ulysses-you] move method
    1558dd9 [ulysses-you] test
    6cd7e57 [ulysses-you] nit
    f8559d2 [ulysses-you] check level at engine discovery
    eedfaee [ulysses-you] split ServiceDiscovery to server and engine
    8b20e6c [ulysses-you] timeout
    0cf524c [ulysses-you] remove unnecessary test
    41d36f9 [ulysses-you] move exception
    eaaa12d [ulysses-you] avoid stop twice
    b801c14 [ulysses-you] fix test
    db39960 [ulysses-you] add start check
    2d1f6dd [ulysses-you] add state check
    e7374aa [ulysses-you] nit
    ddf383c [ulysses-you] fi
    ee088be [ulysses-you] nit
    3c2013b [ulysses-you] nit
    73b386f [ulysses-you] improve conf of test
    2185f49 [ulysses-you] init
    Authored-by: default avatarulysses-you <ulyssesyou18@gmail.com>
    Signed-off-by: default avatarKent Yao <yao@apache.org>
    aa878e4d

免费DevSecOps平台,让您的项目体验完整的DevSecOps流程,让项目更安全