I'm skimming this quickly, but it looks like this would not consider the node locality and the rack locality that the base CombineFileInputFormat class does. Does it mean that we would lose that functionality if we went with this approach? That is a pretty useful feature in reducing traffic.