SPARK-2565. Update ShuffleReadMetrics as blocks are fetched

Maybe we don't care about this...but this results in the metrics reporting that local blocks have been fetched before they're actually read from disk.  Is it too annoying to move this to where the blocks actually get read?