[ 
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

Reply via email to