[ 
https://issues.apache.org/jira/browse/HDFS-16898?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17681735#comment-17681735
 ] 

ASF GitHub Bot commented on HDFS-16898:
---------------------------------------

hfutatzhanghb opened a new pull request, #5330:
URL: https://github.com/apache/hadoop/pull/5330

   Now in method processCommandFromActor,  we have code like below:
   
   ```java
   writeLock();
   try {
     if (actor == bpServiceToActive) {
       return processCommandFromActive(cmd, actor);
     } else {
       return processCommandFromStandby(cmd, actor);
     }
   } finally {
     writeUnlock();
   } 
   ```
   
   if method processCommandFromActive costs much time, the write lock would not 
release.
   
   
   It maybe block the `updateActorStatesFromHeartbeat` method in 
offerService,furthermore, it can cause the lastcontact of datanode very high, 
even dead when lastcontact beyond 600s.
   
   ```java
   bpos.updateActorStatesFromHeartbeat(
       this, resp.getNameNodeHaState());
   ```
   
   here we can make write lock fine-grain in processCommandFromActor method to 
address this problem




> Make write lock fine-grain in processCommandFromActor method
> ------------------------------------------------------------
>
>                 Key: HDFS-16898
>                 URL: https://issues.apache.org/jira/browse/HDFS-16898
>             Project: Hadoop HDFS
>          Issue Type: Improvement
>    Affects Versions: 3.3.4
>            Reporter: ZhangHB
>            Priority: Major
>
> Now in method processCommandFromActor,  we have code like below:
>  
> {code:java}
> writeLock();
> try {
>   if (actor == bpServiceToActive) {
>     return processCommandFromActive(cmd, actor);
>   } else {
>     return processCommandFromStandby(cmd, actor);
>   }
> } finally {
>   writeUnlock();
> } {code}
> if method processCommandFromActive costs much time, the write lock would not 
> release.
>  
> It maybe block the updateActorStatesFromHeartbeat method in 
> offerService,furthermore, it can cause the lastcontact of datanode very high, 
> even dead when lastcontact beyond 600s.
> {code:java}
> bpos.updateActorStatesFromHeartbeat(
>     this, resp.getNameNodeHaState());{code}
> here we can make write lock fine-grain in processCommandFromActor method to 
> address this problem
>  



--
This message was sent by Atlassian Jira
(v8.20.10#820010)

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to