Skip to content

Commit

Permalink
[KYUUBI #3909][TEST] the new client should work properly when the eng…
Browse files Browse the repository at this point in the history
…ine exits unexpectedly

### _Why are the changes needed?_
close #3909

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

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

Closes #3918 from cxzl25/KYUUBI-3909.

Closes #3909

6dd0436 [sychen] sync

Authored-by: sychen <sychen@ctrip.com>
Signed-off-by: Fu Chen <cfmcgrady@gmail.com>
  • Loading branch information
cxzl25 authored and cfmcgrady committed Dec 7, 2022
1 parent b418f89 commit 749b81d
Showing 1 changed file with 1 addition and 1 deletion.
Expand Up @@ -298,7 +298,7 @@ class KyuubiOperationPerUserSuite
// force kill engine without shutdown hook
exitReq.setStatement("java.lang.Runtime.getRuntime().halt(-1)")
exitReq.setSessionHandle(handle)
exitReq.setRunAsync(true)
exitReq.setRunAsync(false)
client.ExecuteStatement(exitReq)
}
withSessionHandle { (client, handle) =>
Expand Down

0 comments on commit 749b81d

Please sign in to comment.