Aggarwal-Raghav commented on code in PR #6042:
URL: https://github.com/apache/hive/pull/6042#discussion_r2295544065
##########
ql/src/java/org/apache/hadoop/hive/ql/processors/SetProcessor.java:
##########
@@ -272,10 +272,25 @@ public Map<String, String> getHiveVariable() {
}
if (register) {
ss.getOverriddenConfigurations().put(key, value);
+
+ if (metaConfOverlayList.contains(key)) {
+ try (Hive hive = Hive.get(ss.getConf())) {
+ // Resources will be closed automatically
+ hive.close(true);
Review Comment:
This hive.close(true) is required becuase in
**_org.apache.hadoop.hive.metastore.client.BaseMetaStoreClient#BaseMetaStoreClient()_**,
if there are multiple session level modifed configs, only the top one is
considered as post the connection creation conf object is not updated.
--
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: gitbox-unsubscr...@hive.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: gitbox-unsubscr...@hive.apache.org
For additional commands, e-mail: gitbox-h...@hive.apache.org
