Fixed log statement and comment

Author: otterc

common/network-shuffle/src/main/java/org/apache/spark/network/shuffle/RemoteBlockPushResolver.java

@@ -885,7 +885,7 @@ void resetChunkTracker() {
      */
     void updateChunkInfo(long chunkOffset, int mapIndex) throws IOException {
       try {
-        logger.trace("{} shuffleId {} reduceId {} updated index current {} updated {}",
+        logger.trace("{} shuffleId {} reduceId {} index current {} updated {}",
           appShuffleId.appId, appShuffleId.shuffleId, reduceId, this.lastChunkOffset, chunkOffset);
         if (indexMetaUpdateFailed) {
           indexFile.getChannel().position(indexFile.getPos());
@@ -894,7 +894,7 @@ void updateChunkInfo(long chunkOffset, int mapIndex) throws IOException {
         // Chunk bitmap should be written to the meta file after the index file because if there are
         // any exceptions during writing the offset to the index file, meta file should not be
         // updated. If the update to the index file is successful but the update to meta file isn't
-        // then the index file position is reset in the catch clause.
+        // then the index file position is not updated.
         writeChunkTracker(mapIndex);
         indexFile.updatePos(8);
         this.lastChunkOffset = chunkOffset;

common/network-shuffle/src/test/java/org/apache/spark/network/shuffle/RemoteBlockPushResolverSuite.java

@@ -451,7 +451,6 @@ public void testRecoverIndexFileAfterIOExceptions() throws IOException {
   @Test
   public void testRecoverIndexFileAfterIOExceptionsInFinalize() throws IOException {
     useTestFiles(true, false);
-    registerExecutor(TEST_APP, prepareLocalDirs(localDirs));
     RemoteBlockPushResolver.PushBlockStreamCallback callback1 =
       (RemoteBlockPushResolver.PushBlockStreamCallback) pushResolver.receiveBlockDataAsStream(
         new PushBlockStream(TEST_APP, 0, 0, 0, 0));