Skip to content

Commit

Permalink
[KYUUBI #2450][FOLLOWUP] Remove opHandle from opHandleSet when except…
Browse files Browse the repository at this point in the history
…ion occurs

### _Why are the changes needed?_

Follow up #2450

In PR #2452, I add opHandle to opHandleSet before running, when running exception we need to remove opHandle from opHandleSet.

### _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

- [ ] [Run test](https://kyuubi.apache.org/docs/latest/develop_tools/testing.html#running-tests) locally before make a pull request

Closes #2504 from wForget/KYUUBI-2450-1.

Closes #2450

a4ef89a [wforget] [KYUUBI-2450][FOLLOWUP] Remove Operation handle from opHandleSet when exception occurs

Authored-by: wforget <643348094@qq.com>
Signed-off-by: Fei Wang <fwang12@ebay.com>
  • Loading branch information
wForget authored and turboFei committed Apr 29, 2022
1 parent fbe38de commit a2c0f78
Showing 1 changed file with 1 addition and 0 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -95,6 +95,7 @@ abstract class AbstractSession(
opHandle
} catch {
case e: KyuubiSQLException =>
opHandleSet.remove(operation.getHandle)
sessionManager.operationManager.closeOperation(operation.getHandle)
throw e
}
Expand Down

0 comments on commit a2c0f78

Please sign in to comment.