[req-changes] Removed _default_client and cache_key, set _current_client on AtlanClient method calls

- Also fixes mypy violations.

Author: Aryamanz29

pyatlan/cache/abstract_asset_cache.py

@@ -4,30 +4,28 @@
 
 import threading
 from abc import ABC, abstractmethod
-from typing import Any
+from typing import TYPE_CHECKING, Any, Dict
 
 from pyatlan.errors import ErrorCode
 from pyatlan.model.assets import Asset
 from pyatlan.model.enums import AtlanConnectorType
 
+if TYPE_CHECKING:
+    from pyatlan.client.atlan import AtlanClient
+
 
 class AbstractAssetCache(ABC):
     """
     Base class for reusable components that are common
     to all caches, where a cache is populated entry-by-entry.
     """
 
-    def __init__(self, client):
+    def __init__(self, client: AtlanClient):
         self.client = client
         self.lock = threading.Lock()
-        self.name_to_guid = dict()
-        self.guid_to_asset = dict()
-        self.qualified_name_to_guid = dict()
-
-    @classmethod
-    @abstractmethod
-    def get_cache(cls):
-        """Abstract method to retreive cache."""
+        self.name_to_guid: Dict[str, str] = dict()
+        self.guid_to_asset: Dict[str, Asset] = dict()
+        self.qualified_name_to_guid: Dict[str, str] = dict()
 
     @abstractmethod
     def lookup_by_guid(self, guid: str):
@@ -84,9 +82,9 @@ def cache(self, asset: Asset):
         name = asset and self.get_name(asset)
         if not all([name, asset.guid, asset.qualified_name]):
             return
-        self.name_to_guid[name] = asset.guid
-        self.guid_to_asset[asset.guid] = asset
-        self.qualified_name_to_guid[asset.qualified_name] = asset.guid
+        self.name_to_guid[name] = asset.guid  # type: ignore[index]
+        self.guid_to_asset[asset.guid] = asset  # type: ignore[index]
+        self.qualified_name_to_guid[asset.qualified_name] = asset.guid  # type: ignore[index]
 
     def _get_by_guid(self, guid: str, allow_refresh: bool = True):
         """

pyatlan/cache/atlan_tag_cache.py

@@ -2,7 +2,7 @@
 # Copyright 2025 Atlan Pte. Ltd.
 from __future__ import annotations
 
-from threading import Lock, local
+from threading import Lock
 from typing import TYPE_CHECKING, Dict, Optional, Set
 
 from pyatlan.errors import ErrorCode
@@ -13,7 +13,6 @@
     from pyatlan.client.atlan import AtlanClient
 
 lock: Lock = Lock()
-atlan_tag_cache_tls = local()  # Thread-local storage (TLS)
 
 
 class AtlanTagCache:
@@ -32,61 +31,39 @@ def __init__(self, client: AtlanClient):
         self.map_id_to_source_tags_attr_id: Dict[str, str] = {}
         self.lock: Lock = Lock()
 
-    @classmethod
-    def get_cache(cls, client: Optional[AtlanClient] = None) -> AtlanTagCache:
-        from pyatlan.client.atlan import AtlanClient
-
-        with lock:
-            client = client or AtlanClient.get_default_client()
-            cache_key = client.cache_key
-
-            if not hasattr(atlan_tag_cache_tls, "caches"):
-                atlan_tag_cache_tls.caches = {}
-
-            if cache_key not in atlan_tag_cache_tls.caches:
-                cache_instance = AtlanTagCache(client=client)
-                cache_instance._refresh_cache()  # Refresh on new cache instance
-                atlan_tag_cache_tls.caches[cache_key] = cache_instance
-
-            return atlan_tag_cache_tls.caches[cache_key]
-
-    @classmethod
-    def refresh_cache(cls) -> None:
+    def refresh_cache(self) -> None:
         """
         Refreshes the cache of Atlan tags by requesting the full set of Atlan tags from Atlan.
         """
-        cls.get_cache()._refresh_cache()
+        self._refresh_cache()
 
-    @classmethod
-    def get_id_for_name(cls, name: str) -> Optional[str]:
+    def get_id_for_name(self, name: str) -> Optional[str]:
         """
         Translate the provided human-readable Atlan tag name to its Atlan-internal ID string.
 
         :param name: human-readable name of the Atlan tag
         :returns: Atlan-internal ID string of the Atlan tag
         """
-        return cls.get_cache()._get_id_for_name(name=name)
+        return self._get_id_for_name(name=name)
 
-    @classmethod
-    def get_name_for_id(cls, idstr: str) -> Optional[str]:
+    def get_name_for_id(self, idstr: str) -> Optional[str]:
         """
         Translate the provided Atlan-internal classification ID string to the human-readable Atlan tag name.
 
         :param idstr: Atlan-internal ID string of the Atlan tag
         :returns: human-readable name of the Atlan tag
         """
-        return cls.get_cache()._get_name_for_id(idstr=idstr)
+        return self._get_name_for_id(idstr=idstr)
 
-    @classmethod
-    def get_source_tags_attr_id(cls, id: str) -> Optional[str]:
+    def get_source_tags_attr_id(self, id: str) -> Optional[str]:
         """
         Translate the provided Atlan-internal Atlan tag ID string to the Atlan-internal name of the attribute that
         captures tag attachment details (for source-synced tags).
 
         :param id: Atlan-internal ID string of the Atlan tag
         :returns: Atlan-internal ID string of the attribute containing source-synced tag attachment details
         """
-        return cls.get_cache()._get_source_tags_attr_id(id)
+        return self._get_source_tags_attr_id(id)
 
     def _refresh_cache(self) -> None:
         """

pyatlan/cache/connection_cache.py

@@ -4,7 +4,6 @@
 
 import logging
 import threading
-from threading import local
 from typing import TYPE_CHECKING, Optional, Union
 
 from pyatlan.cache.abstract_asset_cache import AbstractAssetCache, AbstractAssetName
@@ -17,7 +16,6 @@
     from pyatlan.client.atlan import AtlanClient
 
 lock = threading.Lock()
-connection_cache_tls = local()  # Thread-local storage (TLS)
 LOGGER = logging.getLogger(__name__)
 
 
@@ -44,25 +42,7 @@ class ConnectionCache(AbstractAssetCache):
     def __init__(self, client: AtlanClient):
         super().__init__(client)
 
-    @classmethod
-    def get_cache(cls, client: Optional[AtlanClient] = None) -> ConnectionCache:
-        from pyatlan.client.atlan import AtlanClient
-
-        with lock:
-            client = client or AtlanClient.get_default_client()
-            cache_key = client.cache_key
-
-            if not hasattr(connection_cache_tls, "caches"):
-                connection_cache_tls.caches = {}
-
-            if cache_key not in connection_cache_tls.caches:
-                cache_instance = ConnectionCache(client=client)
-                connection_cache_tls.caches[cache_key] = cache_instance
-
-            return connection_cache_tls.caches[cache_key]
-
-    @classmethod
-    def get_by_guid(cls, guid: str, allow_refresh: bool = True) -> Connection:
+    def get_by_guid(self, guid: str, allow_refresh: bool = True) -> Connection:
         """
         Retrieve a connection from the cache by its UUID.
         If the asset is not found, it will be looked up and added to the cache.
@@ -74,11 +54,10 @@ def get_by_guid(cls, guid: str, allow_refresh: bool = True) -> Connection:
         :raises NotFoundError: if the connection cannot be found (does not exist) in Atlan
         :raises InvalidRequestError: if no UUID was provided for the connection to retrieve
         """
-        return cls.get_cache()._get_by_guid(guid=guid, allow_refresh=allow_refresh)
+        return self._get_by_guid(guid=guid, allow_refresh=allow_refresh)
 
-    @classmethod
     def get_by_qualified_name(
-        cls, qualified_name: str, allow_refresh: bool = True
+        self, qualified_name: str, allow_refresh: bool = True
     ) -> Connection:
         """
         Retrieve a connection from the cache by its unique Atlan-internal name.
@@ -92,13 +71,12 @@ def get_by_qualified_name(
         :raises NotFoundError: if the connection cannot be found (does not exist) in Atlan
         :raises InvalidRequestError: if no qualified_name was provided for the connection to retrieve
         """
-        return cls.get_cache()._get_by_qualified_name(
+        return self._get_by_qualified_name(
             qualified_name=qualified_name, allow_refresh=allow_refresh
         )
 
-    @classmethod
     def get_by_name(
-        cls, name: ConnectionName, allow_refresh: bool = True
+        self, name: ConnectionName, allow_refresh: bool = True
     ) -> Connection:
         """
         Retrieve an connection from the cache by its uniquely identifiable name.
@@ -112,7 +90,7 @@ def get_by_name(
         :raises NotFoundError: if the connection cannot be found (does not exist) in Atlan
         :raises InvalidRequestError: if no name was provided for the connection to retrieve
         """
-        return cls.get_cache()._get_by_name(name=name, allow_refresh=allow_refresh)
+        return self._get_by_name(name=name, allow_refresh=allow_refresh)
 
     def lookup_by_guid(self, guid: str) -> None:
         if not guid:
@@ -149,8 +127,8 @@ def lookup_by_name(self, name: ConnectionName) -> None:
             return
         with self.lock:
             results = self.client.asset.find_connections_by_name(
-                name=name.name,
-                connector_type=name.type,
+                name=name.name,  # type: ignore[arg-type]
+                connector_type=name.type,  # type: ignore[arg-type]
                 attributes=self.SEARCH_ATTRIBUTES,
             )
             if not results: