Avoid configuring root logger in RedisVL (#445)

Closes #443 

This PR updates the logger configuration helper for RedisVL to avoid
configuring the root logger via `logging.basicConfig`. That operation
was interfering with user-configured loggers, causing app logs to be
emitted multiple times and making it trickier to hide RedisVL logs when
running in production.

In its place, we now configure a `NullHandler` (mainly to avoid issues
with loggers not having any handlers configured). **This means that any
RedisVL usage that expects logs to have a specific format must now
configure that format for itself.**

Author: vishal-bala

redisvl/utils/log.py

@@ -1,5 +1,4 @@
 import logging
-import sys
 
 
 def get_logger(name, log_level="info", fmt=None):
@@ -10,13 +9,7 @@ def get_logger(name, log_level="info", fmt=None):
 
     logger = logging.getLogger(name)
 
-    # Only configure this specific logger, not the root logger
-    # Check if the logger already has handlers to respect existing configuration
+    # Add a NullHandler to loggers to avoid "no handler found" warnings
     if not logger.handlers:
-        logging.basicConfig(
-            level=logging.INFO,
-            format="%(asctime)s %(name)s %(levelname)s   %(message)s",
-            datefmt="%H:%M:%S",
-            stream=sys.stdout,
-        )
+        logger.addHandler(logging.NullHandler())
     return logger

tests/unit/logger_interference_checker.py

@@ -9,13 +9,10 @@
     )
 )
 
-# Configure root logger
-root_logger = logging.getLogger()
-root_logger.handlers = [handler]
-root_logger.setLevel(logging.INFO)
-
 # Log before import
 app_logger = logging.getLogger("app")
+app_logger.setLevel(logging.INFO)
+app_logger.addHandler(handler)
 app_logger.info("PRE_IMPORT_FORMAT")
 
 # Import RedisVL

tests/unit/test_utils.py

@@ -489,18 +489,22 @@ def test_logging_configuration_not_overridden(self):
 
         # Extract the log lines
         output_lines = result.stdout.strip().split("\n")
-        pre_import_line = ""
-        post_import_line = ""
+        pre_import_lines = []
+        post_import_lines = []
 
         for line in output_lines:
             if "PRE_IMPORT_FORMAT" in line:
-                pre_import_line = line
+                pre_import_lines.append(line)
             elif "POST_IMPORT_FORMAT" in line:
-                post_import_line = line
+                post_import_lines.append(line)
 
         # Check if we found both lines
-        assert pre_import_line, "No pre-import log message found"
-        assert post_import_line, "No post-import log message found"
+        assert pre_import_lines, "No pre-import log message(s) found"
+        assert post_import_lines, "No post-import log message(s) found"
+        assert len(pre_import_lines) == 1, "Multiple pre-import log messages found"
+        assert len(post_import_lines) == 1, "Multiple post-import log messages found"
+        pre_import_line = pre_import_lines[0]
+        post_import_line = post_import_lines[0]
 
         # Print for debugging
         print(f"Pre-import format: {pre_import_line}")