Github user HyukjinKwon commented on a diff in the pull request:

    https://github.com/apache/spark/pull/19961#discussion_r156555741
  
    --- Diff: 
sql/hive-thriftserver/src/main/java/org/apache/hive/service/cli/operation/ExecuteStatementOperation.java
 ---
    @@ -67,4 +68,16 @@ protected void setConfOverlay(Map<String, String> 
confOverlay) {
           this.confOverlay = confOverlay;
         }
       }
    +
    +  protected void registerCurrentOperationLog() {
    +    if (isOperationLogEnabled) {
    +      if (operationLog == null) {
    +        LOG.warn("Failed to get current OperationLog object of Operation: 
" +
    +                getHandle().getHandleIdentifier());
    --- End diff --
    
    I think the indentation should be double spaced.


---

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to