Reamer opened a new pull request #4020: URL: https://github.com/apache/zeppelin/pull/4020
### What is this PR for? This PR includes: - some code improvements, which are available with JDK 8 - some LOGGER cleanup - remove PrincipalImpl out of ShiroAuthenticationServiceTest.java - use DefaultLdapRealm instead of JndiLdapRealm (see [JavaDoc](https://shiro.apache.org/static/1.7.0/apidocs/org/apache/shiro/realm/ldap/JndiLdapRealm.html)) ### What type of PR is it? Improvement ### What is the Jira issue? * https://issues.apache.org/jira/browse/ZEPPELIN-5198 ### Questions: * Does the licenses files need update? No * Is there breaking changes for older versions? No * Does this needs documentation? No ---------------------------------------------------------------- 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