[CELEBORN-1244] Delete redundant remove operations and handle timeout…

… requests in final check

### What changes were proposed in this pull request?
Delete redundant remove operations and handle timeout requests in final check

### Why are the changes needed?

Delete redundant remove operations and handle timeout requests in final check

### Does this PR introduce _any_ user-facing change?

NO

### How was this patch tested?

PASS GA

Closes #2251 from jiaoqingbo/CELEBORN-1244.

Authored-by: jiaoqingbo <1178404354@qq.com>
Signed-off-by: SteNicholas <programgeek@163.com>

client/src/main/scala/org/apache/celeborn/client/commit/CommitHandler.scala

@@ -357,7 +357,6 @@ abstract class CommitHandler(
             logError(
               s"Ask worker($worker) CommitFiles for $shuffleId failed because of Timeout" +
                 s" (attempt ${status.retriedTimes}/$maxRetries), will not retry.")
-            iter.remove()
           }
         }
       }