vikramahuja1001 commented on code in PR #5303: URL: https://github.com/apache/hive/pull/5303#discussion_r1672347998
########## standalone-metastore/metastore-server/src/main/java/org/apache/hadoop/hive/metastore/security/TokenStoreDelegationTokenSecretManager.java: ########## @@ -238,20 +248,33 @@ public synchronized void stopThreads() { */ protected void removeExpiredTokens() { long now = System.currentTimeMillis(); - Iterator<DelegationTokenIdentifier> i = tokenStore.getAllDelegationTokenIdentifiers() - .iterator(); - while (i.hasNext()) { - DelegationTokenIdentifier id = i.next(); + for (DelegationTokenIdentifier id : tokenStore.getAllDelegationTokenIdentifiers()) { if (now > id.getMaxDate()) { + LOGGER.info("Expiry Thread removing expired token: " + id); this.tokenStore.removeToken(id); // no need to look at token info } else { // get token info to check renew date - DelegationTokenInformation tokenInfo = tokenStore.getToken(id); - if (tokenInfo != null) { - if (now > tokenInfo.getRenewDate()) { - this.tokenStore.removeToken(id); - } + renewIfRequired(now, id, tokenStore.getToken(id)); Review Comment: I have put the renew call in a try-catch statement. This will make sure that for loop is run for all the tokens -- 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. To unsubscribe, e-mail: gitbox-unsubscr...@hive.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: gitbox-unsubscr...@hive.apache.org For additional commands, e-mail: gitbox-h...@hive.apache.org