mohammadjkhan commented on a change in pull request #6972: Support LDAP
authentication/authorization
URL: https://github.com/apache/incubator-druid/pull/6972#discussion_r313418907
##########
File path:
extensions-core/druid-basic-security/src/main/java/org/apache/druid/security/basic/authentication/db/cache/CoordinatorPollingBasicAuthenticatorCacheManager.java
##########
@@ -113,26 +116,51 @@ public void start()
() -> {
try {
long randomDelay = ThreadLocalRandom.current().nextLong(0,
commonCacheConfig.getMaxRandomDelay());
- LOG.debug("Inserting random polling delay of [%s] ms",
randomDelay);
+ LOG.debug("Inserting cachedUserMaps random polling delay of [%s]
ms", randomDelay);
Thread.sleep(randomDelay);
- LOG.debug("Scheduled cache poll is running");
+ LOG.debug("Scheduled user cache poll is running");
for (String authenticatorPrefix : authenticatorPrefixes) {
Map<String, BasicAuthenticatorUser> userMap =
fetchUserMapFromCoordinator(authenticatorPrefix, false);
if (userMap != null) {
cachedUserMaps.put(authenticatorPrefix, userMap);
}
}
- LOG.debug("Scheduled cache poll is done");
+ LOG.debug("Scheduled user cache poll is done");
}
catch (Throwable t) {
LOG.makeAlert(t, "Error occured while polling for
cachedUserMaps.").emit();
}
}
);
+ ScheduledExecutors.scheduleWithFixedDelay(
Review comment:
this is now irrelevant since we aren't polling for configs anymore, code has
be removed
----------------------------------------------------------------
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
With regards,
Apache Git Services
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscr...@druid.apache.org
For additional commands, e-mail: commits-h...@druid.apache.org
