[ https://issues.apache.org/jira/browse/HADOOP-3935?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Johan Oskarsson updated HADOOP-3935: ------------------------------------ Attachment: HADOOP-3935.patch * I have moved PacketResponder to BlockReceiver. * Changed the classes to use the DataNode logger. * Changed public method comments to javadocs * The reason for renaming DataXceiveServer to DataXceiveServer is simply because I assumed it was a typo since the other class is called DataXceiver. * I left Throttler more or less as is since it is being used by multiple classes and a test outside of the package. As mentioned it is fairly generic and could possibly be used by other classes. > Extract classes from DataNode.java > ---------------------------------- > > Key: HADOOP-3935 > URL: https://issues.apache.org/jira/browse/HADOOP-3935 > Project: Hadoop Core > Issue Type: Improvement > Components: dfs > Reporter: Johan Oskarsson > Assignee: Johan Oskarsson > Priority: Trivial > Attachments: HADOOP-3935.patch, HADOOP-3935.patch > > > DataNode.java is becoming hard to navigate with over 3000 lines of code. I > suggest moving some of the classes out into their own files in the same > package. This will also make it easier to see how the classes depend on each > other and to see what code belongs where. -- This message is automatically generated by JIRA. - You can reply to this email to add a comment to the issue online.