[HDFS-7775] Use consistent naming for NN-internal quota related types and functions

• Type: Sub-task
• Status: Closed
• Priority: Minor
• Resolution: Fixed
• Fix Version/s: 2.7.0
• Component/s: datanode, namenode
• Hadoop Flags: Reviewed
• Assignee: Xiaoyu Yao
• Reporter: Xiaoyu Yao
• Created: 11/Feb/15 07:28
• Updated: 24/Apr/15 23:22
• Resolved: 14/Feb/15 05:09

Description

This is based on the code review feedback from arpitagarwal on HDFS-7723. We decide to put the refactoring/renaming work in a separate JIRA to avoid further increasing the size of current patch for HDFS-7723. Below is the initial refactoring to-do list.

• QuotaCounts: typeSpaces and typeCounts are used interchangeably. We should probably name them consistently.

• You renamed DEFAULT_DISKSPACE_QUOTA to DEFAULT_SPACE_QUOTA. DEFAULT_STORAGE_SPACE_QUOTA would be easier to understand. Comments like @return the namespace and diskspace consumed can also be edited to use storage space instead of diskspace.

• Add Javadocs to FsImageFormat.java on why storage type quota for FSImageFormat.java. "We don't have quota by storage type record in legacy fsimages, which is handled in this FSImageFormat class (used by legacy_oiv only). The current fsimage processing is done in the protobuf based class FSImageFormatProtobuf, which is updated with typeCounts by HDFS-7723."