z1131392774 commented on issue #3723: URL: https://github.com/apache/kyuubi/issues/3723#issuecomment-3032100081
Hi @pan3793 @yaooqinn ,sorry to bother you. Following up on my previous comment, I'd like to take over this inactive issue. I have some initial thoughts on the implementation and would appreciate your feedback. Here are my preliminary ideas: 1. SessionState Definition I propose creating a new SessionState enum, similar to OperationState. My initial thought is to include the following states: - INITIALIZED: Session is initialized. - RUNNING: Session is active and running operations. - PENDING: Session is pending, waiting for the engine to start. - CLOSED: Session is closed by a user request. - TIMEOUT: Session is closed due to inactivity timeout. - ERROR: Session terminated with an error. Are these states sufficient, or do you have other suggestions? 2. Fields to Persist for Session Metadata To store session metadata in the MetadataStore, I think we should persist the following core fields, which are essential for management and troubleshooting: - Session Identity: identifier, sessionType - User & Connection: realUser, username, ipAddress - State & Timestamps: state, createTime, endTime - Kyuubi Instance: kyuubiInstance - Configuration: requestConf - Engine Details: engineType, engineId, engineName, engineUrl, engineError - Session/Job Name: requestName Does this list seem complete? Is there anything to add or remove? I will wait for your feedback before starting the work. By the way, should I open a new issue for this implementation, or is it fine to continue the discussion here? Thanks! -- 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-unsubscr...@kyuubi.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: notifications-unsubscr...@kyuubi.apache.org For additional commands, e-mail: notifications-h...@kyuubi.apache.org
