[ https://issues.apache.org/jira/browse/HADOOP-10732?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14059553#comment-14059553 ]
Junping Du commented on HADOOP-10732: ------------------------------------- I look at the patch again. May be your previous way is better and we should also fix org.apache.hadoop.crypto.key.JavaKeyStoreProvider to move the writerLock inside of innerSetCredential() which make lock more fine-grain. Thoughts? > Update without holding write lock in JavaKeyStoreProvider#innerSetCredential() > ------------------------------------------------------------------------------ > > Key: HADOOP-10732 > URL: https://issues.apache.org/jira/browse/HADOOP-10732 > Project: Hadoop Common > Issue Type: Bug > Reporter: Ted Yu > Assignee: Ted Yu > Priority: Minor > Attachments: hadoop-10732-v1.txt > > > In > hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/security/alias/JavaKeyStoreProvider.java, > innerSetCredential() doesn't wrap update with writeLock.lock() / > writeLock.unlock(). -- This message was sent by Atlassian JIRA (v6.2#6252)