castor-extractor 0.9.2__tar.gz → 0.10.0__tar.gz

{castor_extractor-0.9.2 → castor_extractor-0.10.0}/CHANGELOG.md

@@ -1,5 +1,9 @@
 # Changelog
 
+## 0.10.0 - 2023-11-28
+
+* Looker : extract all Looker Explores, even if unused in Dashboards
+
 ## 0.9.2 - 2023-11-23
 
 * Looker : remove deprecated all_looks parameter

{castor_extractor-0.9.2 → castor_extractor-0.10.0}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.1
 Name: castor-extractor
-Version: 0.9.2
+Version: 0.10.0
 Summary: Extract your metadata assets.
 Home-page: https://www.castordoc.com/
 License: EULA

castor_extractor-0.10.0/castor_extractor/visualization/looker/__init__.py

@@ -0,0 +1,3 @@
+from .api import ApiClient, Credentials, lookml_explore_names
+from .assets import LookerAsset
+from .extract import extract_all, iterate_all_data

castor_extractor-0.10.0/castor_extractor/visualization/looker/api/__init__.py

@@ -0,0 +1,3 @@
+from .client import ApiClient
+from .sdk import Credentials
+from .utils import lookml_explore_names

{castor_extractor-0.9.2 → castor_extractor-0.10.0}/castor_extractor/visualization/looker/api/client.py

@@ -191,7 +191,7 @@ class ApiClient:
     def explores(
         self,
         explore_names=Iterator[Tuple[str, str]],
-    ) -> List[LookmlModelExplore]:
+    ) -> Iterator[LookmlModelExplore]:
         """Iterates explores of the given Looker account for the provided model/explore names"""
 
         @safe_mode(self._safe_mode)
@@ -202,11 +202,10 @@ class ApiClient:
             self._on_api_call()
             return explore
 
-        explores = [
-            _call(model_name, explore_name)
-            for model_name, explore_name in explore_names
-        ]
-        return list(filter(None, explores))
+        for lookml_model_name, lookml_explore_name_ in explore_names:
+            explore_ = _call(lookml_model_name, lookml_explore_name_)
+            if explore_ is not None:
+                yield explore_
 
     def connections(self) -> List[DBConnection]:
         """Lists databases connections of the given Looker account"""

castor_extractor-0.10.0/castor_extractor/visualization/looker/api/utils.py

@@ -0,0 +1,24 @@
+from typing import Iterable, Set, Tuple
+
+from .sdk import LookmlModel
+
+
+def lookml_explore_names(
+    lookmls: Iterable[LookmlModel],
+) -> Set[Tuple[str, str]]:
+    """
+    Explores from the lookml models
+    Only valid explores are yielded: with all infos
+    """
+    model_explores = (
+        (model, explore)
+        for model in lookmls
+        for explore in model.explores or []
+    )
+
+    return {
+        (model.name, explore.name)
+        for model, explore in model_explores
+        # accept hidden resources
+        if model.name and explore.name
+    }

{castor_extractor-0.9.2 → castor_extractor-0.10.0}/castor_extractor/visualization/looker/extract.py

@@ -11,12 +11,8 @@ from ...utils import (
     write_json,
     write_summary,
 )
-from .api import (
-    ApiClient,
-    Credentials,
-    dashboard_explore_names,
-    explore_names_associated_to_dashboards,
-)
+from .api import ApiClient, Credentials, lookml_explore_names
+from .api.sdk import LookmlModel
 from .assets import LookerAsset
 from .multithreading import MultithreadingFetcher
 from .parameters import get_parameters
@@ -24,6 +20,15 @@ from .parameters import get_parameters
 logger = logging.getLogger(__name__)
 
 
+def _extract_explores_by_name(
+    lookmls: Iterable[LookmlModel], client: ApiClient
+) -> Iterable[dict]:
+    explore_names = lookml_explore_names(lookmls)
+    explores = client.explores(explore_names)
+    for explore in explores:
+        yield deep_serialize(explore)  # type: ignore
+
+
 def _safe_mode(directory: str) -> SafeMode:
     add_logging_file_handler(directory)
     return SafeMode((Exception,), float("inf"))
@@ -79,10 +84,8 @@ def iterate_all_data(
     if search_per_folder:
         dashboards_stream = fetcher.fetch_assets(LookerAsset.DASHBOARDS)
         yield LookerAsset.DASHBOARDS, StreamableList(dashboards_stream)
-        dashboard_explore_names_ = fetcher.explores
     else:
         dashboards = client.dashboards()
-        dashboard_explore_names_ = dashboard_explore_names(dashboards)
         yield LookerAsset.DASHBOARDS, deep_serialize(dashboards)
 
     logger.info("Extracting lookml models from Looker API")
@@ -90,10 +93,8 @@ def iterate_all_data(
     yield LookerAsset.LOOKML_MODELS, deep_serialize(lookmls)
 
     logger.info("Extracting explores from Looker API")
-    explore_names = explore_names_associated_to_dashboards(
-        lookmls, dashboard_explore_names_
-    )
-    yield LookerAsset.EXPLORES, deep_serialize(client.explores(explore_names))
+    explores = _extract_explores_by_name(lookmls, client)
+    yield LookerAsset.EXPLORES, StreamableList(explores)
     del lookmls
 
     logger.info("Extracting connections from Looker API")

{castor_extractor-0.9.2 → castor_extractor-0.10.0}/castor_extractor/visualization/looker/multithreading.py

@@ -2,13 +2,13 @@ import logging
 import sys
 from concurrent.futures import ThreadPoolExecutor
 from functools import partial
-from typing import Iterable, List, Set, Tuple
+from typing import Iterable, List, Set
 
 from tqdm import tqdm  # type: ignore
 
 from ...utils import RetryStrategy, deep_serialize, retry
-from . import ApiClient, dashboard_explore_names
-from .api.sdk import Dashboard, SDKError
+from . import ApiClient
+from .api.sdk import SDKError
 from .assets import LookerAsset
 
 logger = logging.getLogger(__name__)
@@ -54,16 +54,6 @@ class MultithreadingFetcher:
         self._thread_pool_size = thread_pool_size
         self._log_to_stdout = log_to_stdout
 
-        self.explores: Set[Tuple[str, str]] = set()
-
-    def _save_explore_names(self, dashboards_per_folder: Iterable[Dashboard]):
-        """
-        Since dashboards are streamed right to the file, we need to keep
-        the relevant information to extract Explores later.
-        """
-        explores = dashboard_explore_names(dashboards_per_folder)
-        self.explores.update(explores)
-
     def _progress_bar(self, fetch_results: Iterable, total: int) -> tqdm:
         """Create a tqdm progress bar with the appropriate logs destination"""
         file = sys.stderr
@@ -73,7 +63,7 @@ class MultithreadingFetcher:
 
         return tqdm(fetch_results, total=total, file=file)
 
-    def fetch_assets(self, asset: LookerAsset):
+    def fetch_assets(self, asset: LookerAsset) -> Iterable[dict]:
         """
         Yields serialized Looks or Dashboards with a request per folder ID.
         Requests are parallelised.
@@ -88,12 +78,10 @@ class MultithreadingFetcher:
             fetch_results = executor.map(_fetch, self._folder_ids)
 
             for results in self._progress_bar(fetch_results, total_folders):
-                if asset == LookerAsset.DASHBOARDS:
-                    self._save_explore_names(results)
-
                 for result in results:
                     if not result:
                         continue
+
                     total_assets_count += len(result)
                     yield deep_serialize(result)
 

{castor_extractor-0.9.2 → castor_extractor-0.10.0}/pyproject.toml

@@ -3,7 +3,7 @@ requires = ["setuptools>=61.2"]
 
 [tool.poetry]
 name = "castor-extractor"
-version = "0.9.2"
+version = "0.10.0"
 description = "Extract your metadata assets."
 authors = ["Castor <support@castordoc.com>"]
 license = "EULA"

castor_extractor-0.9.2/castor_extractor/visualization/looker/__init__.py

@@ -1,9 +0,0 @@
-from .api import (
-    ApiClient,
-    Credentials,
-    dashboard_explore_names,
-    explore_names_associated_to_dashboards,
-    lookml_explore_names,
-)
-from .assets import LookerAsset
-from .extract import extract_all, iterate_all_data

castor_extractor-0.9.2/castor_extractor/visualization/looker/api/__init__.py

@@ -1,7 +0,0 @@
-from .client import ApiClient
-from .sdk import Credentials
-from .utils import (
-    dashboard_explore_names,
-    explore_names_associated_to_dashboards,
-    lookml_explore_names,
-)

castor_extractor-0.9.2/castor_extractor/visualization/looker/api/utils.py

@@ -1,49 +0,0 @@
-from typing import Iterable, Set, Tuple
-
-from .sdk import Dashboard, LookmlModel
-
-
-def lookml_explore_names(
-    lookmls: Iterable[LookmlModel],
-) -> Set[Tuple[str, str]]:
-    """
-    Explores from the lookml models
-    Only valid explores are yielded: with all infos
-    """
-    model_explores = (
-        (model, explore)
-        for model in lookmls
-        for explore in model.explores or []
-    )
-
-    return {
-        (model.name, explore.name)
-        for model, explore in model_explores
-        # accept hidden resources
-        if model.name and explore.name
-    }
-
-
-def dashboard_explore_names(
-    dashboards: Iterable[Dashboard],
-) -> Set[Tuple[str, str]]:
-    """Explores that appear in dashboards"""
-    elements = (
-        element
-        for dashboard in dashboards
-        for element in dashboard.dashboard_elements or []
-    )
-
-    return {
-        (element.query.model, element.query.view)
-        for element in elements
-        if element.query and element.query.model and element.query.view
-    }
-
-
-def explore_names_associated_to_dashboards(
-    lookmls: Iterable[LookmlModel],
-    dashboard_explore_names_: Set[Tuple[str, str]],
-):
-    """Retrieve only explores that are associated to a looker dashboard"""
-    return lookml_explore_names(lookmls).intersection(dashboard_explore_names_)