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

Reply via email to