You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@kyuubi.apache.org by GitBox <gi...@apache.org> on 2022/08/16 16:52:21 UTC
[GitHub] [incubator-kyuubi] pan3793 opened a new pull request, #3247: Minor clean up Kyuubi JDBC code
pan3793 opened a new pull request, #3247:
URL: https://github.com/apache/incubator-kyuubi/pull/3247
<!--
Thanks for sending a pull request!
Here are some tips for you:
1. If this is your first time, please read our contributor guidelines: https://kyuubi.readthedocs.io/en/latest/community/contributions.html
2. If the PR is related to an issue in https://github.com/apache/incubator-kyuubi/issues, add '[KYUUBI #XXXX]' in your PR title, e.g., '[KYUUBI #XXXX] Your PR title ...'.
3. If the PR is unfinished, add '[WIP]' in your PR title, e.g., '[WIP][KYUUBI #XXXX] Your PR title ...'.
-->
### _Why are the changes needed?_
<!--
Please clarify why the changes are needed. For instance,
1. If you add a feature, you can talk about the use case of it.
2. If you fix a bug, you can clarify why it is a bug.
-->
Remove used classes `HiveBaseChar`, `HiveChar`, `HiveVarchar`
Pull out `JdbcConnectionParams` from `Utils`
### _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
--
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: notifications-unsubscribe@kyuubi.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@kyuubi.apache.org
For additional commands, e-mail: notifications-help@kyuubi.apache.org
[GitHub] [incubator-kyuubi] cxzl25 commented on a diff in pull request #3247: Minor clean up Kyuubi JDBC code
Posted by GitBox <gi...@apache.org>.
cxzl25 commented on code in PR #3247:
URL: https://github.com/apache/incubator-kyuubi/pull/3247#discussion_r947437701
##########
kyuubi-hive-jdbc/src/main/java/org/apache/kyuubi/jdbc/hive/JdbcConnectionParams.java:
##########
@@ -0,0 +1,226 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one or more
+ * contributor license agreements. See the NOTICE file distributed with
+ * this work for additional information regarding copyright ownership.
+ * The ASF licenses this file to You under the Apache License, Version 2.0
+ * (the "License"); you may not use this file except in compliance with
+ * the License. You may obtain a copy of the License at
+ *
+ * http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ */
+
+package org.apache.kyuubi.jdbc.hive;
+
+import java.util.ArrayList;
+import java.util.LinkedHashMap;
+import java.util.List;
+import java.util.Map;
+
+public class JdbcConnectionParams {
+ // Note on client side parameter naming convention:
+ // Prefer using a shorter camelCase param name instead of using the same name as the
+ // corresponding
+ // HiveServer2 config.
+ // For a jdbc url: jdbc:hive2://<host>:<port>/dbName;sess_var_list?hive_conf_list#hive_var_list,
+ // client side params are specified in sess_var_list
+
+ // Client param names:
+
+ // Retry setting
+ static final String RETRIES = "retries";
+
+ public static final String AUTH_TYPE = "auth";
+ // We're deprecating this variable's name.
+ public static final String AUTH_QOP_DEPRECATED = "sasl.qop";
+ public static final String AUTH_QOP = "saslQop";
+ public static final String AUTH_SIMPLE = "noSasl";
+ public static final String AUTH_USER = "user";
+ public static final String AUTH_PRINCIPAL = "principal";
+ public static final String AUTH_KYUUBI_CLIENT_PRINCIPAL = "kyuubiClientPrincipal";
+ public static final String AUTH_KYUUBI_CLIENT_KEYTAB = "kyuubiClientKeytab";
+ public static final String AUTH_PASSWD = "password";
+ public static final String AUTH_KERBEROS_AUTH_TYPE = "kerberosAuthType";
+ public static final String AUTH_KERBEROS_AUTH_TYPE_FROM_SUBJECT = "fromSubject";
+ public static final String ANONYMOUS_USER = "anonymous";
+ public static final String ANONYMOUS_PASSWD = "anonymous";
+ public static final String USE_SSL = "ssl";
+ public static final String SSL_TRUST_STORE = "sslTrustStore";
+ public static final String SSL_TRUST_STORE_PASSWORD = "trustStorePassword";
+ // We're deprecating the name and placement of this in the parsed map (from hive conf vars to
+ // hive session vars).
+ static final String TRANSPORT_MODE_DEPRECATED = "hive.server2.transport.mode";
+ public static final String TRANSPORT_MODE = "transportMode";
+ // We're deprecating the name and placement of this in the parsed map (from hive conf vars to
+ // hive session vars).
+ static final String HTTP_PATH_DEPRECATED = "hive.server2.thrift.http.path";
+ public static final String HTTP_PATH = "httpPath";
+ public static final String SERVICE_DISCOVERY_MODE = "serviceDiscoveryMode";
+ public static final String PROPERTY_DRIVER = "driver";
+ public static final String PROPERTY_URL = "url";
+ // Don't use dynamic service discovery
+ static final String SERVICE_DISCOVERY_MODE_NONE = "none";
+ // Use ZooKeeper for indirection while using dynamic service discovery
+ static final String SERVICE_DISCOVERY_MODE_ZOOKEEPER = "zooKeeper";
+ static final String ZOOKEEPER_NAMESPACE = "zooKeeperNamespace";
+ // Default namespace value on ZooKeeper.
+ // This value is used if the param "zooKeeperNamespace" is not specified in the JDBC Uri.
+ static final String ZOOKEEPER_DEFAULT_NAMESPACE = "hiveserver2";
+ static final String COOKIE_AUTH = "cookieAuth";
+ static final String COOKIE_NAME = "cookieName";
+ // The default value of the cookie name when CookieAuth=true
+ static final String DEFAULT_COOKIE_NAMES_HS2 = "hive.server2.auth";
+ // The http header prefix for additional headers which have to be appended to the request
+ static final String HTTP_HEADER_PREFIX = "http.header.";
+ // Set the fetchSize
+ static final String FETCH_SIZE = "fetchSize";
+ static final String INIT_FILE = "initFile";
+ static final String WM_POOL = "wmPool";
+ // Cookie prefix
+ static final String HTTP_COOKIE_PREFIX = "http.cookie.";
+
+ static final String CONNECT_TIMEOUT = "connectTimeout";
+ static final String SOCKET_TIMEOUT = "socketTimeout";
+
+ // We support ways to specify application name modeled after some existing DBs, since
+ // there's no standard approach.
+ // MSSQL: applicationName
+ // https://docs.microsoft.com/en-us/sql/connect/jdbc/building-the-connection-url
+ // Postgres 9~: ApplicationName https://jdbc.postgresql.org/documentation/91/connect.html
+ // Note: various ODBC names used include "Application Name", "APP", etc. Add those?
+ static final String[] APPLICATION = new String[] {"applicationName", "ApplicationName"};
+
+ // --------------- Begin 2 way ssl options -------------------------
+ // Use two way ssl. This param will take effect only when ssl=true
+ static final String USE_TWO_WAY_SSL = "twoWay";
+ static final String TRUE = "true";
+ static final String SSL_KEY_STORE = "sslKeyStore";
+ static final String SSL_KEY_STORE_PASSWORD = "keyStorePassword";
+ static final String SSL_KEY_STORE_TYPE = "JKS";
+ static final String SUNX509_ALGORITHM_STRING = "SunX509";
+ static final String SUNJSSE_ALGORITHM_STRING = "SunJSSE";
+ // --------------- End 2 way ssl options ----------------------------
+
+ // Non-configurable params:
+ // Currently supports JKS keystore format
+ static final String SSL_TRUST_STORE_TYPE = "JKS";
+
+ static final String HIVE_VAR_PREFIX = "hivevar:";
+ static final String HIVE_CONF_PREFIX = "hiveconf:";
+ private String host = null;
+ private int port = 0;
+ private String jdbcUriString;
+ private String dbName = Utils.DEFAULT_DATABASE;
+ private Map<String, String> hiveConfs = new LinkedHashMap<>();
+ private Map<String, String> hiveVars = new LinkedHashMap<>();
+ private Map<String, String> sessionVars = new LinkedHashMap<>();
+ private String suppliedURLAuthority;
+ private String zooKeeperEnsemble = null;
+ private String currentHostZnodePath;
+ private final List<String> rejectedHostZnodePaths = new ArrayList<>();
+
+ public JdbcConnectionParams() {}
+
+ public JdbcConnectionParams(JdbcConnectionParams params) {
+ this.host = params.host;
+ this.port = params.port;
+ this.jdbcUriString = params.jdbcUriString;
+ this.dbName = params.dbName;
+ this.hiveConfs.putAll(params.hiveConfs);
+ this.hiveVars.putAll(params.hiveVars);
+ this.sessionVars.putAll(params.sessionVars);
+ this.suppliedURLAuthority = params.suppliedURLAuthority;
+ this.zooKeeperEnsemble = params.zooKeeperEnsemble;
+ this.currentHostZnodePath = params.currentHostZnodePath;
+ this.rejectedHostZnodePaths.addAll(rejectedHostZnodePaths);
Review Comment:
```java
this.rejectedHostZnodePaths.addAll(params.rejectedHostZnodePaths);
```
--
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: notifications-unsubscribe@kyuubi.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@kyuubi.apache.org
For additional commands, e-mail: notifications-help@kyuubi.apache.org
[GitHub] [incubator-kyuubi] pan3793 closed pull request #3247: Minor clean up Kyuubi JDBC code
Posted by GitBox <gi...@apache.org>.
pan3793 closed pull request #3247: Minor clean up Kyuubi JDBC code
URL: https://github.com/apache/incubator-kyuubi/pull/3247
--
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: notifications-unsubscribe@kyuubi.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@kyuubi.apache.org
For additional commands, e-mail: notifications-help@kyuubi.apache.org
[GitHub] [incubator-kyuubi] cxzl25 commented on a diff in pull request #3247: Minor clean up Kyuubi JDBC code
Posted by GitBox <gi...@apache.org>.
cxzl25 commented on code in PR #3247:
URL: https://github.com/apache/incubator-kyuubi/pull/3247#discussion_r947439621
##########
kyuubi-hive-jdbc/src/main/java/org/apache/kyuubi/jdbc/hive/JdbcConnectionParams.java:
##########
@@ -0,0 +1,226 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one or more
+ * contributor license agreements. See the NOTICE file distributed with
+ * this work for additional information regarding copyright ownership.
+ * The ASF licenses this file to You under the Apache License, Version 2.0
+ * (the "License"); you may not use this file except in compliance with
+ * the License. You may obtain a copy of the License at
+ *
+ * http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ */
+
+package org.apache.kyuubi.jdbc.hive;
+
+import java.util.ArrayList;
+import java.util.LinkedHashMap;
+import java.util.List;
+import java.util.Map;
+
+public class JdbcConnectionParams {
+ // Note on client side parameter naming convention:
+ // Prefer using a shorter camelCase param name instead of using the same name as the
+ // corresponding
+ // HiveServer2 config.
+ // For a jdbc url: jdbc:hive2://<host>:<port>/dbName;sess_var_list?hive_conf_list#hive_var_list,
+ // client side params are specified in sess_var_list
+
+ // Client param names:
+
+ // Retry setting
+ static final String RETRIES = "retries";
+
+ public static final String AUTH_TYPE = "auth";
+ // We're deprecating this variable's name.
+ public static final String AUTH_QOP_DEPRECATED = "sasl.qop";
+ public static final String AUTH_QOP = "saslQop";
+ public static final String AUTH_SIMPLE = "noSasl";
+ public static final String AUTH_USER = "user";
+ public static final String AUTH_PRINCIPAL = "principal";
+ public static final String AUTH_KYUUBI_CLIENT_PRINCIPAL = "kyuubiClientPrincipal";
+ public static final String AUTH_KYUUBI_CLIENT_KEYTAB = "kyuubiClientKeytab";
+ public static final String AUTH_PASSWD = "password";
+ public static final String AUTH_KERBEROS_AUTH_TYPE = "kerberosAuthType";
+ public static final String AUTH_KERBEROS_AUTH_TYPE_FROM_SUBJECT = "fromSubject";
+ public static final String ANONYMOUS_USER = "anonymous";
+ public static final String ANONYMOUS_PASSWD = "anonymous";
+ public static final String USE_SSL = "ssl";
+ public static final String SSL_TRUST_STORE = "sslTrustStore";
+ public static final String SSL_TRUST_STORE_PASSWORD = "trustStorePassword";
+ // We're deprecating the name and placement of this in the parsed map (from hive conf vars to
+ // hive session vars).
+ static final String TRANSPORT_MODE_DEPRECATED = "hive.server2.transport.mode";
+ public static final String TRANSPORT_MODE = "transportMode";
+ // We're deprecating the name and placement of this in the parsed map (from hive conf vars to
+ // hive session vars).
+ static final String HTTP_PATH_DEPRECATED = "hive.server2.thrift.http.path";
+ public static final String HTTP_PATH = "httpPath";
+ public static final String SERVICE_DISCOVERY_MODE = "serviceDiscoveryMode";
+ public static final String PROPERTY_DRIVER = "driver";
+ public static final String PROPERTY_URL = "url";
+ // Don't use dynamic service discovery
+ static final String SERVICE_DISCOVERY_MODE_NONE = "none";
+ // Use ZooKeeper for indirection while using dynamic service discovery
+ static final String SERVICE_DISCOVERY_MODE_ZOOKEEPER = "zooKeeper";
+ static final String ZOOKEEPER_NAMESPACE = "zooKeeperNamespace";
+ // Default namespace value on ZooKeeper.
+ // This value is used if the param "zooKeeperNamespace" is not specified in the JDBC Uri.
+ static final String ZOOKEEPER_DEFAULT_NAMESPACE = "hiveserver2";
+ static final String COOKIE_AUTH = "cookieAuth";
+ static final String COOKIE_NAME = "cookieName";
+ // The default value of the cookie name when CookieAuth=true
+ static final String DEFAULT_COOKIE_NAMES_HS2 = "hive.server2.auth";
+ // The http header prefix for additional headers which have to be appended to the request
+ static final String HTTP_HEADER_PREFIX = "http.header.";
+ // Set the fetchSize
+ static final String FETCH_SIZE = "fetchSize";
+ static final String INIT_FILE = "initFile";
+ static final String WM_POOL = "wmPool";
+ // Cookie prefix
+ static final String HTTP_COOKIE_PREFIX = "http.cookie.";
+
+ static final String CONNECT_TIMEOUT = "connectTimeout";
+ static final String SOCKET_TIMEOUT = "socketTimeout";
+
+ // We support ways to specify application name modeled after some existing DBs, since
+ // there's no standard approach.
+ // MSSQL: applicationName
+ // https://docs.microsoft.com/en-us/sql/connect/jdbc/building-the-connection-url
+ // Postgres 9~: ApplicationName https://jdbc.postgresql.org/documentation/91/connect.html
+ // Note: various ODBC names used include "Application Name", "APP", etc. Add those?
+ static final String[] APPLICATION = new String[] {"applicationName", "ApplicationName"};
+
+ // --------------- Begin 2 way ssl options -------------------------
+ // Use two way ssl. This param will take effect only when ssl=true
+ static final String USE_TWO_WAY_SSL = "twoWay";
+ static final String TRUE = "true";
+ static final String SSL_KEY_STORE = "sslKeyStore";
+ static final String SSL_KEY_STORE_PASSWORD = "keyStorePassword";
+ static final String SSL_KEY_STORE_TYPE = "JKS";
+ static final String SUNX509_ALGORITHM_STRING = "SunX509";
+ static final String SUNJSSE_ALGORITHM_STRING = "SunJSSE";
+ // --------------- End 2 way ssl options ----------------------------
+
+ // Non-configurable params:
+ // Currently supports JKS keystore format
+ static final String SSL_TRUST_STORE_TYPE = "JKS";
+
+ static final String HIVE_VAR_PREFIX = "hivevar:";
+ static final String HIVE_CONF_PREFIX = "hiveconf:";
+ private String host = null;
+ private int port = 0;
+ private String jdbcUriString;
+ private String dbName = Utils.DEFAULT_DATABASE;
+ private Map<String, String> hiveConfs = new LinkedHashMap<>();
+ private Map<String, String> hiveVars = new LinkedHashMap<>();
+ private Map<String, String> sessionVars = new LinkedHashMap<>();
+ private String suppliedURLAuthority;
+ private String zooKeeperEnsemble = null;
+ private String currentHostZnodePath;
+ private final List<String> rejectedHostZnodePaths = new ArrayList<>();
+
+ public JdbcConnectionParams() {}
+
+ public JdbcConnectionParams(JdbcConnectionParams params) {
+ this.host = params.host;
+ this.port = params.port;
+ this.jdbcUriString = params.jdbcUriString;
+ this.dbName = params.dbName;
+ this.hiveConfs.putAll(params.hiveConfs);
+ this.hiveVars.putAll(params.hiveVars);
+ this.sessionVars.putAll(params.sessionVars);
+ this.suppliedURLAuthority = params.suppliedURLAuthority;
+ this.zooKeeperEnsemble = params.zooKeeperEnsemble;
+ this.currentHostZnodePath = params.currentHostZnodePath;
+ this.rejectedHostZnodePaths.addAll(rejectedHostZnodePaths);
Review Comment:
Hive master also have similar problem.
https://github.com/apache/hive/blob/master/jdbc/src/java/org/apache/hive/jdbc/Utils.java#L232
```java
this.rejectedHostZnodePaths.addAll(rejectedHostZnodePaths);
```
--
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: notifications-unsubscribe@kyuubi.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@kyuubi.apache.org
For additional commands, e-mail: notifications-help@kyuubi.apache.org
[GitHub] [incubator-kyuubi] pan3793 commented on pull request #3247: Minor clean up Kyuubi JDBC code
Posted by GitBox <gi...@apache.org>.
pan3793 commented on PR #3247:
URL: https://github.com/apache/incubator-kyuubi/pull/3247#issuecomment-1217451477
Thanks, merging to master
--
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: notifications-unsubscribe@kyuubi.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@kyuubi.apache.org
For additional commands, e-mail: notifications-help@kyuubi.apache.org
[GitHub] [incubator-kyuubi] codecov-commenter commented on pull request #3247: Minor clean up Kyuubi JDBC code
Posted by GitBox <gi...@apache.org>.
codecov-commenter commented on PR #3247:
URL: https://github.com/apache/incubator-kyuubi/pull/3247#issuecomment-1216970040
# [Codecov](https://codecov.io/gh/apache/incubator-kyuubi/pull/3247?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation) Report
> Merging [#3247](https://codecov.io/gh/apache/incubator-kyuubi/pull/3247?src=pr&el=desc&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation) (b6e8d37) into [master](https://codecov.io/gh/apache/incubator-kyuubi/commit/e584880f750d8dda8107e725a7a4daedc2fbc056?el=desc&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation) (e584880) will **increase** coverage by `0.21%`.
> The diff coverage is `29.88%`.
```diff
@@ Coverage Diff @@
## master #3247 +/- ##
============================================
+ Coverage 50.92% 51.14% +0.21%
Complexity 13 13
============================================
Files 469 467 -2
Lines 26123 26031 -92
Branches 3611 3605 -6
============================================
+ Hits 13304 13313 +9
+ Misses 11545 11448 -97
+ Partials 1274 1270 -4
```
| [Impacted Files](https://codecov.io/gh/apache/incubator-kyuubi/pull/3247?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation) | Coverage Δ | |
|---|---|---|
| [...n/java/org/apache/hive/beeline/KyuubiCommands.java](https://codecov.io/gh/apache/incubator-kyuubi/pull/3247/diff?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation#diff-a3l1dWJpLWhpdmUtYmVlbGluZS9zcmMvbWFpbi9qYXZhL29yZy9hcGFjaGUvaGl2ZS9iZWVsaW5lL0t5dXViaUNvbW1hbmRzLmphdmE=) | `0.00% <ø> (ø)` | |
| [.../java/org/apache/kyuubi/jdbc/KyuubiHiveDriver.java](https://codecov.io/gh/apache/incubator-kyuubi/pull/3247/diff?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation#diff-a3l1dWJpLWhpdmUtamRiYy9zcmMvbWFpbi9qYXZhL29yZy9hcGFjaGUva3l1dWJpL2pkYmMvS3l1dWJpSGl2ZURyaXZlci5qYXZh) | `4.87% <ø> (ø)` | |
| [.../org/apache/kyuubi/jdbc/hive/KyuubiConnection.java](https://codecov.io/gh/apache/incubator-kyuubi/pull/3247/diff?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation#diff-a3l1dWJpLWhpdmUtamRiYy9zcmMvbWFpbi9qYXZhL29yZy9hcGFjaGUva3l1dWJpL2pkYmMvaGl2ZS9LeXV1YmlDb25uZWN0aW9uLmphdmE=) | `4.16% <ø> (ø)` | |
| [.../org/apache/kyuubi/jdbc/hive/KyuubiDataSource.java](https://codecov.io/gh/apache/incubator-kyuubi/pull/3247/diff?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation#diff-a3l1dWJpLWhpdmUtamRiYy9zcmMvbWFpbi9qYXZhL29yZy9hcGFjaGUva3l1dWJpL2pkYmMvaGl2ZS9LeXV1YmlEYXRhU291cmNlLmphdmE=) | `0.00% <ø> (ø)` | |
| [...c/main/java/org/apache/kyuubi/jdbc/hive/Utils.java](https://codecov.io/gh/apache/incubator-kyuubi/pull/3247/diff?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation#diff-a3l1dWJpLWhpdmUtamRiYy9zcmMvbWFpbi9qYXZhL29yZy9hcGFjaGUva3l1dWJpL2pkYmMvaGl2ZS9VdGlscy5qYXZh) | `36.78% <0.00%> (-0.11%)` | :arrow_down: |
| [...he/kyuubi/jdbc/hive/ZooKeeperHiveClientHelper.java](https://codecov.io/gh/apache/incubator-kyuubi/pull/3247/diff?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation#diff-a3l1dWJpLWhpdmUtamRiYy9zcmMvbWFpbi9qYXZhL29yZy9hcGFjaGUva3l1dWJpL2pkYmMvaGl2ZS9ab29LZWVwZXJIaXZlQ2xpZW50SGVscGVyLmphdmE=) | `3.19% <0.00%> (+0.19%)` | :arrow_up: |
| [.../apache/kyuubi/jdbc/hive/JdbcConnectionParams.java](https://codecov.io/gh/apache/incubator-kyuubi/pull/3247/diff?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation#diff-a3l1dWJpLWhpdmUtamRiYy9zcmMvbWFpbi9qYXZhL29yZy9hcGFjaGUva3l1dWJpL2pkYmMvaGl2ZS9KZGJjQ29ubmVjdGlvblBhcmFtcy5qYXZh) | `48.14% <48.14%> (ø)` | |
| [...n/scala/org/apache/kyuubi/engine/ProcBuilder.scala](https://codecov.io/gh/apache/incubator-kyuubi/pull/3247/diff?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation#diff-a3l1dWJpLXNlcnZlci9zcmMvbWFpbi9zY2FsYS9vcmcvYXBhY2hlL2t5dXViaS9lbmdpbmUvUHJvY0J1aWxkZXIuc2NhbGE=) | `82.60% <0.00%> (-0.63%)` | :arrow_down: |
| [...g/apache/kyuubi/operation/BatchJobSubmission.scala](https://codecov.io/gh/apache/incubator-kyuubi/pull/3247/diff?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation#diff-a3l1dWJpLXNlcnZlci9zcmMvbWFpbi9zY2FsYS9vcmcvYXBhY2hlL2t5dXViaS9vcGVyYXRpb24vQmF0Y2hKb2JTdWJtaXNzaW9uLnNjYWxh) | `74.40% <0.00%> (ø)` | |
| [...ain/scala/org/apache/kyuubi/engine/EngineRef.scala](https://codecov.io/gh/apache/incubator-kyuubi/pull/3247/diff?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation#diff-a3l1dWJpLXNlcnZlci9zcmMvbWFpbi9zY2FsYS9vcmcvYXBhY2hlL2t5dXViaS9lbmdpbmUvRW5naW5lUmVmLnNjYWxh) | `75.00% <0.00%> (+0.92%)` | :arrow_up: |
| ... and [4 more](https://codecov.io/gh/apache/incubator-kyuubi/pull/3247/diff?src=pr&el=tree-more&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation) | |
:mega: We’re building smart automated test selection to slash your CI/CD build times. [Learn more](https://about.codecov.io/iterative-testing/?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation)
--
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: notifications-unsubscribe@kyuubi.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@kyuubi.apache.org
For additional commands, e-mail: notifications-help@kyuubi.apache.org