Merge branch 'remove_force_dump_hook' into 'master'

remove force dump hook See merge request data/monolith!2121 GitOrigin-RevId: 596bfb6336211d6673f8fdc85d533c319ad5dadd
roddylan · Sep 6, 2023 · 5990d14 · 5990d14
1 parent d69893a
commit 5990d14
Showing 1 changed file with 0 additions and 5 deletions.
diff --git a/monolith/native_training/cpu_training.py b/monolith/native_training/cpu_training.py
@@ -1208,11 +1208,6 @@ def create_saver():
           use_native_multi_hash_table=self.config.use_native_multi_hash_table,
           no_first_save=not should_do_first_save)
 
-      if self.config.enable_sync_training and self.config.enable_realtime_training:
-        hooks.append(
-            sync_training_hooks.SyncTrainingForceDumpHook(
-                model_dir, saver_hook.timer))
-
       hooks.append(saver_hook)
 
       if not self.config.enable_sync_training: