[CELEBORN-805][FOLLOWUP] Remove unnecessary TODO

### What changes were proposed in this pull request? As title ### Why are the changes needed? cleanup the unnecessary TODO which introduced in #1727 ### Does this PR introduce _any_ user-facing change? No ### How was this patch tested? Review Closes #1728 from cfmcgrady/shutdown. Authored-by: Fu Chen <[email protected]> Signed-off-by: Cheng Pan <[email protected]> (cherry picked from commit 16d9c65) Signed-off-by: Cheng Pan <[email protected]>
apache · Jul 18, 2023 · 1aecf34 · 1aecf34
1 parent affc8f1
commit 1aecf34
Showing 1 changed file with 0 additions and 1 deletion.
diff --git a/worker/src/main/scala/org/apache/celeborn/service/deploy/worker/Worker.scala b/worker/src/main/scala/org/apache/celeborn/service/deploy/worker/Worker.scala
@@ -438,7 +438,6 @@ private[celeborn] class Worker(
       masterClient.close()
       replicateServer.shutdown(graceful)
       fetchServer.shutdown(graceful)
-      // TODO: `pushServer` never be closed before this PR.
       pushServer.shutdown(graceful)
 
       super.shutdown(graceful)