zjffdu commented on a change in pull request #4007:
URL: https://github.com/apache/zeppelin/pull/4007#discussion_r549759258
##########
File path:
zeppelin-interpreter/src/main/java/org/apache/zeppelin/interpreter/KerberosInterpreter.java
##########
@@ -136,6 +136,8 @@ public Object call() throws Exception {
logger.info("Ran runKerberosLogin command successfully.");
kinitFailCount = 0;
// schedule another kinit run with a fixed delay.
+ logger.info("Scheduling Kerberos ticket refresh thread with interval
" +
Review comment:
Could you help to rename `logger` to `LOGGER` ? It is our convention for
naming static variables.
----------------------------------------------------------------
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:
[email protected]
