Minor Description
BlockManager#DEFAULT_MAP_LOAD_FACTOR is deadcode. It no longer does anything since blocks are now stored in a GSet whose size is fixed.
BlocksMap#blocks does not need to be volatile. Whenever it is accessed, it is accessed under the FSNamesystem lock. Furthermore, access to this data structure is not thread-safe.