belugabehr commented on a change in pull request #1946: URL: https://github.com/apache/hive/pull/1946#discussion_r576861619
########## File path: service/src/java/org/apache/hive/service/cli/thrift/ThriftCLIService.java ########## @@ -530,10 +600,9 @@ public TCloseSessionResp CloseSession(TCloseSessionReq req) throws TException { SessionHandle sessionHandle = new SessionHandle(req.getSessionHandle()); cliService.closeSession(sessionHandle); resp.setStatus(OK_STATUS); - ThriftCLIServerContext context = - (ThriftCLIServerContext)currentServerContext.get(); + ThriftCLIServerContext context = (ThriftCLIServerContext) currentServerContext.get(); Review comment: Well, you know me, I try to fix up what I can. There is no need for this line break based on current checkstyle rules, but I can revert. ---------------------------------------------------------------- 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