[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 apache#1727 ### Does this PR introduce _any_ user-facing change? No ### How was this patch tested? Review Closes apache#1728 from cfmcgrady/shutdown. Authored-by: Fu Chen <[email protected]> Signed-off-by: Cheng Pan <[email protected]>
GH-Gloway · Jul 18, 2023 · 16d9c65 · 16d9c65
1 parent 7c6644b
commit 16d9c65
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)