[GitHub] [flink] zentol commented on a change in pull request #12616: [FLINK-17977] Decrease log clutter

[GitBox <gi...@apache.org>]

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:
users@infra.apache.org