You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@kyuubi.apache.org by "pan3793 (via GitHub)" <gi...@apache.org> on 2023/02/14 10:09:15 UTC

[GitHub] [kyuubi] pan3793 opened a new issue, #4327: [Bug] Keep session/operation id consistent between server and engine

pan3793 opened a new issue, #4327:
URL: https://github.com/apache/kyuubi/issues/4327

   ### Code of Conduct
   
   - [X] I agree to follow this project's [Code of Conduct](https://www.apache.org/foundation/policies/conduct)
   
   
   ### Search before asking
   
   - [X] I have searched in the [issues](https://github.com/apache/kyuubi/issues?q=is%3Aissue) and found no similar issues.
   
   
   ### Describe the bug
   
   This is a regression because it was supported in #1032(only for session id) but broken after #1346, we need to fix this issue to reduce troubleshoot effort between server and engine.
   
   ### Affects Version(s)
   
   master/1.7/1.6
   
   ### Kyuubi Server Log Output
   
   _No response_
   
   ### Kyuubi Engine Log Output
   
   _No response_
   
   ### Kyuubi Server Configurations
   
   _No response_
   
   ### Kyuubi Engine Configurations
   
   _No response_
   
   ### Additional context
   
   Before #1346, the open session process is synchronized,
   ```
   client ===open session===> server ===open session===> engine
   ```
   so the server can wait for the engine to return a remote session id, then reuse it to keep the session id be consistent. This is actually the way proposed by #1032.
   
   #1346 changed the open session to async, so we need to reverse the above approach, to let the server generate a session id and pass it to the engine for reuse. This approach is also suited for operation id.
   
   ### Are you willing to submit PR?
   
   - [ ] Yes. I would be willing to submit a PR with guidance from the Kyuubi community to fix.
   - [ ] No. I cannot submit a PR at this time.


-- 
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: notifications-unsubscribe@kyuubi.apache.org.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@kyuubi.apache.org
For additional commands, e-mail: notifications-help@kyuubi.apache.org

[GitHub] [kyuubi] turboFei commented on issue #4327: [Bug] Keep session/operation id consistent between server and engine

Posted by "turboFei (via GitHub)" <gi...@apache.org>.
turboFei commented on issue #4327:
URL: https://github.com/apache/kyuubi/issues/4327#issuecomment-1445006950

   In #4412 , we align the server/engine session identifier.


-- 
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: notifications-unsubscribe@kyuubi.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@kyuubi.apache.org
For additional commands, e-mail: notifications-help@kyuubi.apache.org

[GitHub] [kyuubi] turboFei commented on issue #4327: [Bug] Keep session/operation id consistent between server and engine

Posted by "turboFei (via GitHub)" <gi...@apache.org>.
turboFei commented on issue #4327:
URL: https://github.com/apache/kyuubi/issues/4327#issuecomment-1445354137

   #4415 Align the server/engine ExecuteStatement operation handle for Spark engine


-- 
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: notifications-unsubscribe@kyuubi.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@kyuubi.apache.org
For additional commands, e-mail: notifications-help@kyuubi.apache.org

[GitHub] [kyuubi] turboFei commented on issue #4327: [Bug] Keep session/operation id consistent between server and engine

Posted by "turboFei (via GitHub)" <gi...@apache.org>.
turboFei commented on issue #4327:
URL: https://github.com/apache/kyuubi/issues/4327#issuecomment-1445009599

   It is not possible to keep operation id consitent for all operation type.
   
   Now, we can only support:
   - ExecuteStatement


-- 
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: notifications-unsubscribe@kyuubi.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@kyuubi.apache.org
For additional commands, e-mail: notifications-help@kyuubi.apache.org