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

chengpan 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 89497c4  [KYUUBI #2204] Make comments consistent with code in EngineRef
89497c4 is described below

commit 89497c41ec1bdadbd07afc175088993c17553c2b
Author: jiaoqingbo <[email protected]>
AuthorDate: Thu Mar 24 11:47:23 2022 +0800

    [KYUUBI #2204] Make comments consistent with code in EngineRef
    
    ### _Why are the changes needed?_
    
    fix #2204
    
    ### _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 #2205 from jiaoqingbo/2204.
    
    Closes #2204
    
    418613e4 [jiaoqingbo] change {KYUUBI_VERSION} to version
    217d7be7 [jiaoqingbo] [KYUUBI #2204] Make comments consistent with code in 
EngineRef
    
    Authored-by: jiaoqingbo <[email protected]>
    Signed-off-by: Cheng Pan <[email protected]>
---
 .../src/main/scala/org/apache/kyuubi/engine/EngineRef.scala       | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git 
a/kyuubi-server/src/main/scala/org/apache/kyuubi/engine/EngineRef.scala 
b/kyuubi-server/src/main/scala/org/apache/kyuubi/engine/EngineRef.scala
index 9294257..e0b0a1c 100644
--- a/kyuubi-server/src/main/scala/org/apache/kyuubi/engine/EngineRef.scala
+++ b/kyuubi-server/src/main/scala/org/apache/kyuubi/engine/EngineRef.scala
@@ -123,13 +123,13 @@ private[kyuubi] class EngineRef(
    * The EngineSpace used to expose itself to the KyuubiServers in 
`serverSpace`
    *
    * For `CONNECTION` share level:
-   *   /`serverSpace_CONNECTION_engineType`/`user`/`engineRefId`
+   *   /`serverSpace_version_CONNECTION_engineType`/`user`/`engineRefId`
    * For `USER` share level:
-   *   /`serverSpace_USER_engineType`/`user`[/`subdomain`]
+   *   /`serverSpace_version_USER_engineType`/`user`[/`subdomain`]
    * For `GROUP` share level:
-   *   /`serverSpace_GROUP_engineType`/`primary group name`[/`subdomain`]
+   *   /`serverSpace_version_GROUP_engineType`/`primary group 
name`[/`subdomain`]
    * For `SERVER` share level:
-   *   /`serverSpace_SERVER_engineType`/`kyuubi server user`[/`subdomain`]
+   *   /`serverSpace_version_SERVER_engineType`/`kyuubi server 
user`[/`subdomain`]
    */
   @VisibleForTesting
   private[kyuubi] lazy val engineSpace: String = {

Reply via email to