[ https://issues.apache.org/jira/browse/HDFS-12754?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16254237#comment-16254237 ]
Kuhu Shukla commented on HDFS-12754: ------------------------------------ Thank you for the review [~xiaochen]. I will address the test related changes soon, although checkstyle change that remains seems unnecessary as I mentioned earlier. Also, bq. I think we're saying the same thing here: dfsClient.closeAllFilesBeingWritten out of the LeaseRenewer's synchronized block, This is already being done in the v7 patch. The closeAll Files lock is not exercised from within the LeaseRenewer lock. What am I missing here? > Lease renewal can hit a deadlock > --------------------------------- > > Key: HDFS-12754 > URL: https://issues.apache.org/jira/browse/HDFS-12754 > Project: Hadoop HDFS > Issue Type: Bug > Affects Versions: 2.8.1 > Reporter: Kuhu Shukla > Assignee: Kuhu Shukla > Attachments: HDFS-12754.001.patch, HDFS-12754.002.patch, > HDFS-12754.003.patch, HDFS-12754.004.patch, HDFS-12754.005.patch, > HDFS-12754.006.patch, HDFS-12754.007.patch > > > The Client and the renewer can hit a deadlock during close operation since > closeFile() reaches back to the DFSClient#removeFileBeingWritten. This is > possible if the client class close when the renewer is renewing a lease. -- This message was sent by Atlassian JIRA (v6.4.14#64029) --------------------------------------------------------------------- To unsubscribe, e-mail: hdfs-issues-unsubscr...@hadoop.apache.org For additional commands, e-mail: hdfs-issues-h...@hadoop.apache.org