timothy65535 opened a new issue #933:
URL: https://github.com/apache/incubator-kyuubi/issues/933


   ### 1. Description
   
   EngineRef use exists znode to determine whether engine started successfully 
or not(use the last znode), there are inconsistencies in this. So, take an idea 
to improve the detection mechanism by adding the started sessionid to the 
znone. `EngineRef` will check the sessionid in a loop after a engine started.
   
   ### 2. Here are two options
   **Option I, add sesssionid to the data of znode**
   
![image](https://user-images.githubusercontent.com/86483005/128994276-5c9c7839-0a21-4253-8989-110b88ac0e74.png)
   
   **Option II, add sessionid to the path of znode**
   
![image](https://user-images.githubusercontent.com/86483005/129527282-46f7f726-e110-4d87-9272-dda9b9f325bf.png)
   
   ### 3. Solution
   
   In order to be consistent with the design of server znode, prefer to choose 
_Option II_ . 
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


Reply via email to