[HDFS-5665] Remove the unnecessary writeLock while initializing CacheManager in FsNameSystem Ctor

• Type: Improvement
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 2.2.0, 3.0.0-alpha1
• Fix Version/s: 2.3.0
• Component/s: namenode
• Hadoop Flags: Reviewed
• Assignee: Uma Maheswara Rao G
• Reporter: Uma Maheswara Rao G
• Created: 13/Dec/13 09:12
• Updated: 12/May/16 18:15
• Resolved: 16/Dec/13 17:13

Description

I just saw the below piece of code in Fsnamesystem ctor.

 writeLock();
      try {
        this.cacheManager = new CacheManager(this, conf, blockManager);
      } finally {
        writeUnlock();
      }

It seems unnecessary to keep writeLock here. I am not sure if there is a clear reason to keep the lock.