iodone opened a new issue #961: URL: https://github.com/apache/incubator-kyuubi/issues/961
### 1. Describe the bug **Bug Analysis** https://github.com/apache/incubator-kyuubi/blob/4aa93d3cbdb71ca6c8eb91bd426fdd5191d03f02/kyuubi-common/src/main/scala/org/apache/kyuubi/session/AbstractSession.scala#L180-L183 In the implementation of closeOperation, `sessionManager.operationManager.closeOperation(operationHandle)` will be called first. https://github.com/apache/incubator-kyuubi/blob/4aa93d3cbdb71ca6c8eb91bd426fdd5191d03f02/kyuubi-common/src/main/scala/org/apache/kyuubi/operation/OperationManager.scala#L102-L106 In the implementation of `operationManager.closeOperation`, the operation in operationManager is removed first, and then closed, if an exception occurs in the `operation.close` phase, the `opHandleSet.remove(operationHandle)` in `Session.closeOperation.remove(operationHandle)` will not be executed, which will lead to inconsistency between the operation of OperationManager.handleToOperation and Session.opHandleSet <!-- A clear and concise description of what the bug is. --> ### 2. Environments #### 2.1 Versions - Kyuubi - [ ] 1.1.0 - [ ] 1.0.3 - [ ] 0.8.x and earlier <!-- Please update Kyuubi, versions of 0.x are no longer supported by the community --> - Spark - [ ] 3.1.x - [ ] 3.0.x - [ ] 2.4.x and earlier <!-- Please update Spark, versions of 2.x are no longer supported by the community --> #### 2.2 Kyuubi Configurations #### 2.3 Spark Configurations ### 3. Steps to reproduce #### 3.1 Operations #### 3.2 Expected behavior <!-- A clear and concise description of what you expected to happen. --> #### 3.3 Actual behavior ### 4. Additional context #### 4.1 Log of the server #### 4.2 Log of the corresponding 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
