You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@kyuubi.apache.org by ch...@apache.org on 2023/03/07 05:56:01 UTC
[kyuubi] branch branch-1.7 updated: [KYUUBI #4443] Do not set engine session init sql for alive probe session
This is an automated email from the ASF dual-hosted git repository.
chengpan pushed a commit to branch branch-1.7
in repository https://gitbox.apache.org/repos/asf/kyuubi.git
The following commit(s) were added to refs/heads/branch-1.7 by this push:
new 969517f6a [KYUUBI #4443] Do not set engine session init sql for alive probe session
969517f6a is described below
commit 969517f6adfe5a002dfe96ad4c0be4370efa450d
Author: fwang12 <fw...@ebay.com>
AuthorDate: Sat Mar 4 18:41:34 2023 +0800
[KYUUBI #4443] Do not set engine session init sql for alive probe session
### _Why are the changes needed?_
Prevent duplicate queries on UI.
### _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.readthedocs.io/en/master/develop_tools/testing.html#running-tests) locally before make a pull request
Closes #4443 from turboFei/alive_session_conf.
Closes #4443
1ec571643 [fwang12] do not init for alive probe
Authored-by: fwang12 <fw...@ebay.com>
Signed-off-by: fwang12 <fw...@ebay.com>
---
.../main/scala/org/apache/kyuubi/client/KyuubiSyncThriftClient.scala | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/kyuubi-server/src/main/scala/org/apache/kyuubi/client/KyuubiSyncThriftClient.scala b/kyuubi-server/src/main/scala/org/apache/kyuubi/client/KyuubiSyncThriftClient.scala
index e3e66960b..12a4c824c 100644
--- a/kyuubi-server/src/main/scala/org/apache/kyuubi/client/KyuubiSyncThriftClient.scala
+++ b/kyuubi-server/src/main/scala/org/apache/kyuubi/client/KyuubiSyncThriftClient.scala
@@ -186,7 +186,8 @@ class KyuubiSyncThriftClient private (
Utils.tryLogNonFatalError {
req.setConfiguration((configs ++ Map(
KyuubiConf.SESSION_NAME.key -> sessionName,
- KYUUBI_SESSION_HANDLE_KEY -> UUID.randomUUID().toString)).asJava)
+ KYUUBI_SESSION_HANDLE_KEY -> UUID.randomUUID().toString,
+ KyuubiConf.ENGINE_SESSION_INITIALIZE_SQL.key -> "")).asJava)
val resp = aliveProbeClient.OpenSession(req)
ThriftUtils.verifyTStatus(resp.getStatus)
_aliveProbeSessionHandle = resp.getSessionHandle