simplify and address PR feedback

ludwig/utils/logging_utils.py

@@ -1,32 +1,15 @@
-import time
-
 _logged = set()
-_disabled = False
-_periodic_log = False
-_last_logged = 0.0
 
 
-def log_once(key):
+def log_once(key: str) -> bool:
     """Returns True if this is the "first" call for a given key.
 
-    Various logging settings can adjust the definition of "first".
-
     Example:
         if log_once("some_key"):
             logger.info("Some verbose logging statement") # noqa
     """
 
-    global _last_logged
-
-    if _disabled:
-        return False
-    elif key not in _logged:
+    if key not in _logged:
         _logged.add(key)
-        _last_logged = time.time()
         return True
-    elif _periodic_log and time.time() - _last_logged > 60.0:
-        _logged.clear()
-        _last_logged = time.time()
-        return False
-    else:
-        return False
+    return False

ludwig/utils/structural_warning.py

@@ -3,7 +3,7 @@
 from ludwig.utils.logging_utils import log_once
 
 
-def warn_structure_refactor(old_module: str, new_module: str, direct: bool = True):
+def warn_structure_refactor(old_module: str, new_module: str, direct: bool = True) -> None:
     """Create structure refactor warning to indicate modules new location post.
 
     Only creates a warning once per module.