John Zhuge created HDFS-9554: -------------------------------- Summary: Change string modificationLabel to enum in protobuf message SnapshotDiffReportEntryProto Key: HDFS-9554 URL: https://issues.apache.org/jira/browse/HDFS-9554 Project: Hadoop HDFS Issue Type: Improvement Components: hdfs-client, namenode Reporter: John Zhuge Assignee: John Zhuge Priority: Minor
Field modificationLabel represents the type of message SnapshotDiffReportEntryProto, but it gets its value from a different place, a Java enum SnapshotDiffReport.DiffType. There are a few drawbacks: * SnapshotDiffReportEntryProto is not well encapsulated. * The Java user of this message has to have the access to SnapshotDiffReport.DiffType as well. * In language other than Java, users have to duplicate the string values of DiffType. We should change its type from string to enum and rename it to "type". -- This message was sent by Atlassian JIRA (v6.3.4#6332)