Found more redundant searches in BlockManager.addStoredBLock() and BlockManager.getNodes(Block).
The pattern here is that we first call BlocksMap.numNodes(Block) in order to determine the size of the array of data-nodes. This is one search in blocksMap. Then we call BlocksMap.nodeIterator(Block), which also performs a search. This is all not necessary because we already have the block in the form of BlockInfo from blocksMap and do not need to search for it.
|Status||Open [ 1 ]||Patch Available [ 10002 ]|
|Status||Patch Available [ 10002 ]||Resolved [ 5 ]|
|Resolution||Fixed [ 1 ]|
|Status||Resolved [ 5 ]||Closed [ 6 ]|
|Transition||Time In Source Status||Execution Times||Last Executer||Last Execution Date|
|1h 13m||1||Konstantin Shvachko||05/Aug/09 03:35|
|1d 4m||1||Konstantin Shvachko||06/Aug/09 03:40|
|383d 18h 8m||1||Tom White||24/Aug/10 21:48|