fix: refine logging (#70)

Author: yaozheng-fang

veadk/knowledgebase/knowledgebase.py

@@ -32,9 +32,7 @@ def __init__(
         top_k: int = 10,
         db_config=None,
     ):
-        logger.info(
-            f"Initializing knowledgebase: backend={backend} top_k={top_k} db_config={db_config}"
-        )
+        logger.info(f"Initializing knowledgebase: backend={backend} top_k={top_k}")
 
         self.backend = backend
         self.top_k = top_k
@@ -43,7 +41,7 @@ def __init__(
         self.adapter = get_knowledgebase_database_adapter(self.db_client)
 
         logger.info(
-            f"Initialized knowledgebase: db_client={self.db_client} adapter={self.adapter}"
+            f"Initialized knowledgebase: db_client={self.db_client.__class__.__name__} adapter={self.adapter}"
         )
 
     def add(

veadk/memory/long_term_memory.py

@@ -63,7 +63,7 @@ def __init__(
         self.adapter = get_long_term_memory_database_adapter(self.db_client)
 
         logger.info(
-            f"Initialized long term memory: db_client={self.db_client} adapter={self.adapter}"
+            f"Initialized long term memory: db_client={self.db_client.__class__.__name__} adapter={self.adapter}"
         )
 
     def _filter_and_convert_events(self, events: list[Event]) -> list[str]:

veadk/memory/short_term_memory.py

@@ -60,18 +60,14 @@ def __init__(
             self.session_service = InMemorySessionService()
         elif self.backend == "database":
             if self.db_url == "" or self.db_url is None:
-                logger.warning(
-                    f"The `db_url` {self.db_url} is an empty or None string."
-                )
+                logger.warning("The `db_url` is an empty or None string.")
                 self._use_default_database()
             else:
                 try:
                     self.session_service = DatabaseSessionService(db_url=self.db_url)
-                    logger.info("Connected to database.")
+                    logger.info("Connected to database with db_url.")
                 except Exception as e:
-                    logger.error(
-                        f"Failed to connect to database {self.db_url}, error: {e}."
-                    )
+                    logger.error(f"Failed to connect to database, error: {e}.")
                     self._use_default_database()
         else:
             raise ValueError(f"Unknown short term memory backend: {self.backend}")