[jira] [Updated] (HDFS-5207) In BlockPlacementPolicy, update 2 parameters of chooseTarget()
[ https://issues.apache.org/jira/browse/HDFS-5207?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Junping Du updated HDFS-5207: - Attachment: HDFS-5207-v3.patch In BlockPlacementPolicy, update 2 parameters of chooseTarget() -- Key: HDFS-5207 URL: https://issues.apache.org/jira/browse/HDFS-5207 Project: Hadoop HDFS Issue Type: Improvement Components: namenode Reporter: Junping Du Assignee: Junping Du Attachments: HDFS-5207-v1.patch, HDFS-5207-v2.patch, HDFS-5207-v3.patch We should change chooseTarget(..., DatanodeDescriptor writer, ...) to chooseTarget (..., node writer, ...) as the only important property of writer is to identify other nodes' location relationship so more generic one could be better. It also helps to cover cases that client node is not a Datanode. Also, the excluded nodes are not necessary to be a HashMap with the same key and value, Set is more proper in this case. -- This message is automatically generated by JIRA. If you think it was sent incorrectly, please contact your JIRA administrators For more information on JIRA, see: http://www.atlassian.com/software/jira
[jira] [Updated] (HDFS-5207) In BlockPlacementPolicy, update 2 parameters of chooseTarget()
[ https://issues.apache.org/jira/browse/HDFS-5207?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Junping Du updated HDFS-5207: - Attachment: HDFS-5207-v4.patch Sorry. The cast issue should be addressed in v3 patch. Now update in v4 patch. Thanks! In BlockPlacementPolicy, update 2 parameters of chooseTarget() -- Key: HDFS-5207 URL: https://issues.apache.org/jira/browse/HDFS-5207 Project: Hadoop HDFS Issue Type: Improvement Components: namenode Reporter: Junping Du Assignee: Junping Du Attachments: HDFS-5207-v1.patch, HDFS-5207-v2.patch, HDFS-5207-v3.patch, HDFS-5207-v4.patch We should change chooseTarget(..., DatanodeDescriptor writer, ...) to chooseTarget (..., node writer, ...) as the only important property of writer is to identify other nodes' location relationship so more generic one could be better. It also helps to cover cases that client node is not a Datanode. Also, the excluded nodes are not necessary to be a HashMap with the same key and value, Set is more proper in this case. -- This message is automatically generated by JIRA. If you think it was sent incorrectly, please contact your JIRA administrators For more information on JIRA, see: http://www.atlassian.com/software/jira
[jira] [Updated] (HDFS-5207) In BlockPlacementPolicy, update 2 parameters of chooseTarget()
[ https://issues.apache.org/jira/browse/HDFS-5207?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Tsz Wo (Nicholas), SZE updated HDFS-5207: - Hadoop Flags: Reviewed +1 patch looks good. In BlockPlacementPolicy, update 2 parameters of chooseTarget() -- Key: HDFS-5207 URL: https://issues.apache.org/jira/browse/HDFS-5207 Project: Hadoop HDFS Issue Type: Improvement Components: namenode Reporter: Junping Du Assignee: Junping Du Attachments: HDFS-5207-v1.patch, HDFS-5207-v2.patch, HDFS-5207-v3.patch, HDFS-5207-v4.patch We should change chooseTarget(..., DatanodeDescriptor writer, ...) to chooseTarget (..., node writer, ...) as the only important property of writer is to identify other nodes' location relationship so more generic one could be better. It also helps to cover cases that client node is not a Datanode. Also, the excluded nodes are not necessary to be a HashMap with the same key and value, Set is more proper in this case. -- This message is automatically generated by JIRA. If you think it was sent incorrectly, please contact your JIRA administrators For more information on JIRA, see: http://www.atlassian.com/software/jira
[jira] [Updated] (HDFS-5207) In BlockPlacementPolicy, update 2 parameters of chooseTarget()
[ https://issues.apache.org/jira/browse/HDFS-5207?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Tsz Wo (Nicholas), SZE updated HDFS-5207: - Resolution: Fixed Fix Version/s: 2.3.0 Status: Resolved (was: Patch Available) I have committed this. Thanks, Junping! In BlockPlacementPolicy, update 2 parameters of chooseTarget() -- Key: HDFS-5207 URL: https://issues.apache.org/jira/browse/HDFS-5207 Project: Hadoop HDFS Issue Type: Improvement Components: namenode Reporter: Junping Du Assignee: Junping Du Fix For: 2.3.0 Attachments: HDFS-5207-v1.patch, HDFS-5207-v2.patch, HDFS-5207-v3.patch, HDFS-5207-v4.patch We should change chooseTarget(..., DatanodeDescriptor writer, ...) to chooseTarget (..., node writer, ...) as the only important property of writer is to identify other nodes' location relationship so more generic one could be better. It also helps to cover cases that client node is not a Datanode. Also, the excluded nodes are not necessary to be a HashMap with the same key and value, Set is more proper in this case. -- This message is automatically generated by JIRA. If you think it was sent incorrectly, please contact your JIRA administrators For more information on JIRA, see: http://www.atlassian.com/software/jira
[jira] [Updated] (HDFS-5207) In BlockPlacementPolicy, update 2 parameters of chooseTarget()
[ https://issues.apache.org/jira/browse/HDFS-5207?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Junping Du updated HDFS-5207: - Summary: In BlockPlacementPolicy, update 2 parameters of chooseTarget() (was: In BlockPlacementPolicy, the type of writer in parameters of chooseTarget() should be updated from DatanodeDescriptor to Node) In BlockPlacementPolicy, update 2 parameters of chooseTarget() -- Key: HDFS-5207 URL: https://issues.apache.org/jira/browse/HDFS-5207 Project: Hadoop HDFS Issue Type: Improvement Components: namenode Reporter: Junping Du Assignee: Junping Du Attachments: HDFS-5207-v1.patch We should change chooseTarget(..., DatanodeDescriptor writer, ...) to chooseTarget (..., node writer, ...) as the only important property of writer is to identify other nodes' location relationship so more generic one could be better. It also helps to cover cases that client node is not a Datanode also. -- This message is automatically generated by JIRA. If you think it was sent incorrectly, please contact your JIRA administrators For more information on JIRA, see: http://www.atlassian.com/software/jira
[jira] [Updated] (HDFS-5207) In BlockPlacementPolicy, update 2 parameters of chooseTarget()
[ https://issues.apache.org/jira/browse/HDFS-5207?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Junping Du updated HDFS-5207: - Description: We should change chooseTarget(..., DatanodeDescriptor writer, ...) to chooseTarget (..., node writer, ...) as the only important property of writer is to identify other nodes' location relationship so more generic one could be better. It also helps to cover cases that client node is not a Datanode. Also, the excluded nodes are not necessary to be a HashMap with the same key and value, Set is more proper in this case. (was: We should change chooseTarget(..., DatanodeDescriptor writer, ...) to chooseTarget (..., node writer, ...) as the only important property of writer is to identify other nodes' location relationship so more generic one could be better. It also helps to cover cases that client node is not a Datanode also.) In BlockPlacementPolicy, update 2 parameters of chooseTarget() -- Key: HDFS-5207 URL: https://issues.apache.org/jira/browse/HDFS-5207 Project: Hadoop HDFS Issue Type: Improvement Components: namenode Reporter: Junping Du Assignee: Junping Du Attachments: HDFS-5207-v1.patch We should change chooseTarget(..., DatanodeDescriptor writer, ...) to chooseTarget (..., node writer, ...) as the only important property of writer is to identify other nodes' location relationship so more generic one could be better. It also helps to cover cases that client node is not a Datanode. Also, the excluded nodes are not necessary to be a HashMap with the same key and value, Set is more proper in this case. -- This message is automatically generated by JIRA. If you think it was sent incorrectly, please contact your JIRA administrators For more information on JIRA, see: http://www.atlassian.com/software/jira
[jira] [Updated] (HDFS-5207) In BlockPlacementPolicy, update 2 parameters of chooseTarget()
[ https://issues.apache.org/jira/browse/HDFS-5207?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Junping Du updated HDFS-5207: - Attachment: HDFS-5207-v2.patch Fix tiny issues on comments in v2 patch. In BlockPlacementPolicy, update 2 parameters of chooseTarget() -- Key: HDFS-5207 URL: https://issues.apache.org/jira/browse/HDFS-5207 Project: Hadoop HDFS Issue Type: Improvement Components: namenode Reporter: Junping Du Assignee: Junping Du Attachments: HDFS-5207-v1.patch, HDFS-5207-v2.patch We should change chooseTarget(..., DatanodeDescriptor writer, ...) to chooseTarget (..., node writer, ...) as the only important property of writer is to identify other nodes' location relationship so more generic one could be better. It also helps to cover cases that client node is not a Datanode. Also, the excluded nodes are not necessary to be a HashMap with the same key and value, Set is more proper in this case. -- This message is automatically generated by JIRA. If you think it was sent incorrectly, please contact your JIRA administrators For more information on JIRA, see: http://www.atlassian.com/software/jira
