[ https://issues.apache.org/jira/browse/HADOOP-10376?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14026247#comment-14026247 ]
Zesheng Wu commented on HADOOP-10376: ------------------------------------- Hi Chris, The proposal and patch both look great to me, and fix my doubt of why namenode having so many refresh*Protocols. One more minor suggestion: should we mark the old refresh* functions as deprecated? > Refactor refresh*Protocols into a single generic refreshConfigProtocol > ---------------------------------------------------------------------- > > Key: HADOOP-10376 > URL: https://issues.apache.org/jira/browse/HADOOP-10376 > Project: Hadoop Common > Issue Type: Improvement > Reporter: Chris Li > Assignee: Chris Li > Priority: Minor > Attachments: HADOOP-10376.patch, HADOOP-10376.patch, > HADOOP-10376.patch, RefreshFrameworkProposal.pdf > > > See https://issues.apache.org/jira/browse/HADOOP-10285 > There are starting to be too many refresh*Protocols We can refactor them to > use a single protocol with a variable payload to choose what to do. > Thereafter, we can return an indication of success or failure. -- This message was sent by Atlassian JIRA (v6.2#6252)