mustafaiman commented on a change in pull request #1577:
URL: https://github.com/apache/hive/pull/1577#discussion_r509312017
##########
File path: ql/src/java/org/apache/hadoop/hive/ql/DriverUtils.java
##########
@@ -95,7 +95,7 @@ public static SessionState setUpSessionState(HiveConf conf,
String user, boolean
if (sessionState == null) {
// Note: we assume that workers run on the same threads repeatedly, so
we can set up
// the session here and it will be reused without explicitly
storing in the worker.
- sessionState = new SessionState(conf, user);
+ sessionState = new SessionState(conf, user, true);
Review comment:
background threads do not need async delete. Many compaction tests
specifically have sync assumptions. I dont see any benefit in moving background
operations to async cleanup model.
----------------------------------------------------------------
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.
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
