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]

Reply via email to