yaooqinn commented on a change in pull request #1328:
URL: https://github.com/apache/incubator-kyuubi/pull/1328#discussion_r742516432
##########
File path:
kyuubi-server/src/test/scala/org/apache/kyuubi/operation/KyuubiOperationPerConnectionSuite.scala
##########
@@ -65,4 +65,36 @@ class KyuubiOperationPerConnectionSuite extends
WithKyuubiServer with HiveJDBCTe
assert(verboseMessage.contains("Failed to detect the root cause"))
}
}
+
+ test("client sync query cost time longer than engine.request.timeout") {
+ withSessionConf(Map(
+ KyuubiConf.ENGINE_REQUEST_TIMEOUT.key -> "PT5S"
+ ))(Map.empty)(Map.empty) {
+ withSessionHandle { (client, handle) =>
+ val executeStmtReq = new TExecuteStatementReq()
+ executeStmtReq.setStatement("select java_method('java.lang.Thread',
'sleep', 6000L)")
+ executeStmtReq.setSessionHandle(handle)
+ executeStmtReq.setRunAsync(false)
+ val executeStmtResp = client.ExecuteStatement(executeStmtReq)
+ val getOpStatusReq = new
TGetOperationStatusReq(executeStmtResp.getOperationHandle)
+ val getOpStatusResp = client.GetOperationStatus(getOpStatusReq)
+ assert(getOpStatusResp.getStatus.getStatusCode ===
TStatusCode.SUCCESS_STATUS)
Review comment:
why SUCCESS_STATUS？
--
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]
