hadoop-common-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Kan Zhang (JIRA)" <j...@apache.org>
Subject [jira] Commented: (HADOOP-5481) Namenode should return lease recovery request with other requests
Date Thu, 19 Mar 2009 19:22:50 GMT

    [ 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.


Mime
View raw message