[ https://issues.apache.org/jira/browse/HADOOP-17612?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17625951#comment-17625951 ]
ASF GitHub Bot commented on HADOOP-17612: ----------------------------------------- melissayou commented on code in PR #5067: URL: https://github.com/apache/hadoop/pull/5067#discussion_r1008566388 ########## hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/security/token/delegation/ZKDelegationTokenSecretManager.java: ########## @@ -417,24 +408,18 @@ private void processTokenAddOrUpdate(byte[] data) throws IOException { if (numRead > -1) { DelegationTokenInformation tokenInfo = new DelegationTokenInformation(renewDate, password); - synchronized (this) { Review Comment: It was introduced from https://github.com/apache/hadoop/pull/2047/files I don't get the full context of that change but in `AbstractDelegationTokenSecretManager.java`, they changed implementations of `currentTokens` and `allKeys` from `hashmap` (not thread safe) to `concurrentHashMap`(thread safe). That's why they removed `synchronized` as it's a duplicate protection. > Upgrade Zookeeper to 3.6.3 and Curator to 5.2.0 > ----------------------------------------------- > > Key: HADOOP-17612 > URL: https://issues.apache.org/jira/browse/HADOOP-17612 > Project: Hadoop Common > Issue Type: Task > Reporter: Viraj Jasani > Assignee: Viraj Jasani > Priority: Major > Labels: pull-request-available > Fix For: 3.4.0 > > Time Spent: 4h > Remaining Estimate: 0h > > Let's upgrade Zookeeper and Curator to 3.6.3 and 5.2.0 respectively. > Curator 5.2 also supports Zookeeper 3.5 servers. -- This message was sent by Atlassian Jira (v8.20.10#820010) --------------------------------------------------------------------- To unsubscribe, e-mail: common-issues-unsubscr...@hadoop.apache.org For additional commands, e-mail: common-issues-h...@hadoop.apache.org