Github user cloud-fan commented on a diff in the pull request:

    https://github.com/apache/spark/pull/20281#discussion_r162593238
  
    --- Diff: 
sql/hive-thriftserver/src/main/java/org/apache/hive/service/cli/session/HiveSessionImpl.java
 ---
    @@ -223,6 +223,18 @@ private void configureSession(Map<String, String> 
sessionConfMap) throws HiveSQL
     
       @Override
       public void setOperationLogSessionDir(File operationLogRootDir) {
    +    if (!operationLogRootDir.exists()) {
    +      LOG.warn("The operation log root directory is removed, recreating:" +
    --- End diff --
    
    we need an extra space: `... recreating: " +`


---

---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org

Reply via email to