zentol commented on a change in pull request #12616:
URL: https://github.com/apache/flink/pull/12616#discussion_r439287133
##########
File path:
flink-runtime/src/main/java/org/apache/flink/runtime/security/SecurityUtils.java
##########
@@ -98,7 +98,7 @@ static void installContext(SecurityConfiguration config)
throws Exception {
LOG.error("Error occur when
instantiate security context with: " + contextFactoryClass , le);
}
} else {
- LOG.warn("Unable to install
incompatible security context factory {}", contextFactoryClass);
+ LOG.debug("Unable to install security
context factory {}", contextFactoryClass);
Review comment:
The code here cannot differentiate between user-configured and default
factories. If no context could be installed, I think this method will actually
fail (go down a bit, installedContext == null),
----------------------------------------------------------------
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]
