Github user zuotingbing commented on a diff in the pull request:
https://github.com/apache/spark/pull/20025#discussion_r163136632
--- Diff:
sql/hive-thriftserver/src/main/java/org/apache/hive/service/cli/session/SessionManager.java
---
@@ -80,7 +76,6 @@ public synchronized void init(HiveConf hiveConf) {
}
createBackgroundOperationPool();
addService(operationManager);
- super.init(hiveConf);
--- End diff --
yes, we could keep super.init() here and remove initCompositeService in
SparkSQLSessionManager.init(). thanks
---
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]