[ 
https://issues.apache.org/jira/browse/HADOOP-5481?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12683575#action_12683575
 ] 

Kan Zhang commented on HADOOP-5481:
-----------------------------------

I need to add a new type of request, which is combinable with others and 
preferably sent to DN at first opportunity. I could combine my new request with 
the lease recovery request and again with the other 2, but it looks ugly. 
Fundamentally, we need to be able to answer the question of whether the lease 
recovery request is handled independently from the other 2. If yes, we combine 
them. If not, we put a note in the code so that people won't get confused when 
adding new types of requests in the future. Otherwise, we're kind of in the 
dark.

> Namenode should return lease recovery request with other requests
> -----------------------------------------------------------------
>
>                 Key: HADOOP-5481
>                 URL: https://issues.apache.org/jira/browse/HADOOP-5481
>             Project: Hadoop Core
>          Issue Type: Improvement
>          Components: dfs
>            Reporter: Kan Zhang
>            Assignee: Kan Zhang
>         Attachments: HADOOP-5481.patch
>
>
> HADOOP-5034 modified NN to return both replication and deletion requests to 
> DN in one reply to a heartbeat. However, the lease recovery request is still 
> sent separately by itself. Is there a reason for this? If not, I suggest we 
> combine them together. This will make it less confusing when adding new types 
> of requests, which are combinable as well.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.

Reply via email to