diff --git a/hadoop-hdfs-project/hadoop-hdfs/src/main/java/org/apache/hadoop/hdfs/server/blockmanagement/BlockCollection.java b/hadoop-hdfs-project/hadoop-hdfs/src/main/java/org/apache/hadoop/hdfs/server/blockmanagement/BlockCollection.java
index b8805907959..c0dfc140382 100644
--- a/hadoop-hdfs-project/hadoop-hdfs/src/main/java/org/apache/hadoop/hdfs/server/blockmanagement/BlockCollection.java
+++ b/hadoop-hdfs-project/hadoop-hdfs/src/main/java/org/apache/hadoop/hdfs/server/blockmanagement/BlockCollection.java
@@ -32,62 +32,62 @@ public interface BlockCollection {
   /**
    * Get the last block of the collection.
    */
-  public BlockInfo getLastBlock();
+  BlockInfo getLastBlock();
 
   /** 
    * Get content summary.
    */
-  public ContentSummary computeContentSummary(BlockStoragePolicySuite bsps)
+  ContentSummary computeContentSummary(BlockStoragePolicySuite bsps)
       throws AccessControlException;
 
   /**
    * @return the number of blocks or block groups
    */ 
-  public int numBlocks();
+  int numBlocks();
 
   /**
    * Get the blocks (striped or contiguous).
    */
-  public BlockInfo[] getBlocks();
+  BlockInfo[] getBlocks();
 
   /**
    * Get preferred block size for the collection 
    * @return preferred block size in bytes
    */
-  public long getPreferredBlockSize();
+  long getPreferredBlockSize();
 
   /**
    * Get block replication for the collection.
    * @return block replication value. Return 0 if the file is erasure coded.
    */
-  public short getPreferredBlockReplication();
+  short getPreferredBlockReplication();
 
   /**
    * @return the storage policy ID.
    */
-  public byte getStoragePolicyID();
+  byte getStoragePolicyID();
 
   /**
    * Get the name of the collection.
    */
-  public String getName();
+  String getName();
 
   /**
    * Set the block (contiguous or striped) at the given index.
    */
-  public void setBlock(int index, BlockInfo blk);
+  void setBlock(int index, BlockInfo blk);
 
   /**
    * Convert the last block of the collection to an under-construction block
    * and set the locations.
    */
-  public void convertLastBlockToUC(BlockInfo lastBlock,
+  void convertLastBlockToUC(BlockInfo lastBlock,
       DatanodeStorageInfo[] targets) throws IOException;
 
   /**
    * @return whether the block collection is under construction.
    */
-  public boolean isUnderConstruction();
+  boolean isUnderConstruction();
 
   /**
    * @return whether the block collection is in striping format