castor-extractor 0.24.9__py3-none-any.whl → 0.24.10__py3-none-any.whl

CHANGELOG.md

@@ -1,5 +1,9 @@
 # Changelog
 
+## 0.24.10 - 2025-04-30
+
+* Tableau - skip warnings instead of raising an error
+
 ## 0.24.9 - 2025-04-16
 
 * Introduce API client for **Coalesce**

castor_extractor/visualization/tableau/client/client_metadata_api.py

@@ -29,24 +29,62 @@ _RETRY_BASE_MS = 10_000
 _RETRY_COUNT = 4
 
 
+def _is_timeout(error: dict) -> bool:
+    error_message = error.get("message")
+    return error_message == _TIMEOUT_MESSAGE
+
+
+def _is_warning(error: dict) -> bool:
+    extensions = error.get("extensions")
+    if not extensions:
+        return False
+
+    severity = extensions.get("severity")
+    if not severity:
+        return False
+
+    return severity.lower() == "warning"
+
+
 def _check_errors(answer: dict) -> None:
     """
-    handle errors in graphql response:
+    Handle errors in graphql response:
     - return None when there's no errors in the answer
-    - TableauApiTimeout if any of the errors is a timeout
-    - TableauApiError (generic) otherwise
+    - raise TableauApiTimeout if any of the errors is a timeout
+    - else raise TableauApiError if any of the errors is critical
+    - return None otherwise
+    More info about Tableau errors:
+    https://help.tableau.com/current/api/metadata_api/en-us/docs/meta_api_errors.html#other-errors
     """
     if "errors" not in answer:
         return
 
     errors = answer["errors"]
 
-    for error in errors:
-        if error.get("message") == _TIMEOUT_MESSAGE:
-            # we need specific handling for timeout issues (retry strategy)
-            raise TableauApiTimeout(errors)
+    has_timeout_errors = False
+    has_critical_errors = False
 
-    raise TableauApiError(answer["errors"])
+    for error in errors:
+        if _is_timeout(error):
+            has_timeout_errors = True
+            continue
+
+        if _is_warning(error):
+            # in this case, the answer contains the data anyway
+            # just display the warning
+            logger.warning(error)
+            continue
+
+        # at this point, it's not a timeout error
+        # besides, it's not a warning (severity is either "error" or Unknown)
+        has_critical_errors = True
+
+    if has_timeout_errors:
+        raise TableauApiTimeout(errors)
+    if has_critical_errors:
+        raise TableauApiError(errors)
+
+    return None
 
 
 def gql_query_scroll(

castor_extractor/visualization/tableau/client/client_metadata_api_test.py

@@ -1,4 +1,7 @@
-from .client_metadata_api import _deduplicate
+import pytest
+
+from .client_metadata_api import _TIMEOUT_MESSAGE, _check_errors, _deduplicate
+from .errors import TableauApiError, TableauApiTimeout
 
 
 def test__deduplicate():
@@ -29,3 +32,80 @@ def test__deduplicate():
     assert len(deduplicated) == 5
     deduplicated_keys = {item["id"] for item in deduplicated}
     assert deduplicated_keys == {1, 2, 3, 4, 5}
+
+
+def _timeout_error():
+    return {"message": _TIMEOUT_MESSAGE}
+
+
+def _critical_error():
+    return {
+        "message": "critical error",
+        "extensions": {"severity": "Error"},
+    }
+
+
+def _unknown_severity_error():
+    return {
+        "message": "unknown severity error",
+        "extensions": {"severity": None},
+    }
+
+
+def _warning():
+    return {
+        "message": "critical error",
+        "extensions": {"severity": "Warning"},
+    }
+
+
+def test__check_errors():
+    answer = {"data": []}  # no errors
+    assert _check_errors(answer) is None
+
+    # only warnings
+    answer = {
+        "data": [],
+        "errors": [
+            _warning(),
+            _warning(),
+            _warning(),
+        ],
+    }
+    assert _check_errors(answer) is None
+
+    # timeout issues should be prio (because they can be retried)
+    answer = {
+        "data": [],
+        "errors": [
+            _warning(),
+            _critical_error(),
+            _timeout_error(),
+            _warning(),
+        ],
+    }
+    with pytest.raises(TableauApiTimeout):
+        _check_errors(answer)
+
+    # expect critical error
+    answer = {
+        "data": [],
+        "errors": [
+            _warning(),
+            _warning(),
+            _critical_error(),
+        ],
+    }
+    with pytest.raises(TableauApiError):
+        _check_errors(answer)
+
+    # unknown severity is considered as critical
+    answer = {
+        "data": [],
+        "errors": [
+            _unknown_severity_error(),
+            _warning(),
+        ],
+    }
+    with pytest.raises(TableauApiError):
+        _check_errors(answer)

{castor_extractor-0.24.9.dist-info → castor_extractor-0.24.10.dist-info}/METADATA

@@ -1,6 +1,6 @@
 Metadata-Version: 2.1
 Name: castor-extractor
-Version: 0.24.9
+Version: 0.24.10
 Summary: Extract your metadata assets.
 Home-page: https://www.castordoc.com/
 License: EULA
@@ -210,6 +210,10 @@ For any questions or bug report, contact us at [support@castordoc.com](mailto:su
 
 # Changelog
 
+## 0.24.10 - 2025-04-30
+
+* Tableau - skip warnings instead of raising an error
+
 ## 0.24.9 - 2025-04-16
 
 * Introduce API client for **Coalesce**

{castor_extractor-0.24.9.dist-info → castor_extractor-0.24.10.dist-info}/RECORD

@@ -1,4 +1,4 @@
-CHANGELOG.md,sha256=UKD2ldg9s00KOoVfWjnyB_m50R0fnpPLbpmkZHKoOQM,16821
+CHANGELOG.md,sha256=PDbRWM3MtvskjUAjZXatnsZwB2M22ufediw5J3GETVo,16901
 Dockerfile,sha256=xQ05-CFfGShT3oUqaiumaldwA288dj9Yb_pxofQpufg,301
 DockerfileUsage.md,sha256=2hkJQF-5JuuzfPZ7IOxgM6QgIQW7l-9oRMFVwyXC4gE,998
 LICENCE,sha256=sL-IGa4hweyya1HgzMskrRdybbIa2cktzxb5qmUgDg8,8254
@@ -276,8 +276,8 @@ castor_extractor/visualization/tableau/__init__.py,sha256=eFI_1hjdkxyUiAYiy3szwy
 castor_extractor/visualization/tableau/assets.py,sha256=HbCRd8VCj1WBEeqg9jwnygnT7xOFJ6PQD7Lq7sV-XR0,635
 castor_extractor/visualization/tableau/client/__init__.py,sha256=P8RKFKOC63WkH5hdEytJOwHS9vzQ8GXreLfXZetmMP8,78
 castor_extractor/visualization/tableau/client/client.py,sha256=zzqhzIqKyJygo4ZNGk6cZh0e6Z9R1W5T0P9un52KC1M,7626
-castor_extractor/visualization/tableau/client/client_metadata_api.py,sha256=fARj7xroHfMd4nlo5CJK5jPok5UsHznOQpIpNaECVHw,5274
-castor_extractor/visualization/tableau/client/client_metadata_api_test.py,sha256=lbsq5mLtqeNc5EsmCw9Mvl8qcvMsTcJTepHwy1ToyvA,969
+castor_extractor/visualization/tableau/client/client_metadata_api.py,sha256=ryRq4_qUok8vvWGhj5CNWXtwR2JlUsu1qjsov2KhQTE,6286
+castor_extractor/visualization/tableau/client/client_metadata_api_test.py,sha256=rikyQKDLFYHLJhHJTF3LwWhKJ80svtTsYp5n7n9oTU8,2665
 castor_extractor/visualization/tableau/client/client_rest_api.py,sha256=x4dNw4PPJdalTlGowwkANwqiS2ZhGxzpQytkHq3KbpY,3988
 castor_extractor/visualization/tableau/client/client_tsc.py,sha256=VI_PJyd1ty3HSYXHHQjshmG2ziowIbrwJRonRPCHbks,1820
 castor_extractor/visualization/tableau/client/credentials.py,sha256=uQICIgeXmLZfOroTgZt7PuKNKTyqQllRGSTcOmIfrKU,1893
@@ -416,8 +416,8 @@ castor_extractor/warehouse/sqlserver/queries/table.sql,sha256=kbBQP-TdG5px1IVgyx
 castor_extractor/warehouse/sqlserver/queries/user.sql,sha256=gOrZsMVypusR2dc4vwVs4E1a-CliRsr_UjnD2EbXs-A,94
 castor_extractor/warehouse/sqlserver/query.py,sha256=g0hPT-RmeGi2DyenAi3o72cTlQsLToXIFYojqc8E5fQ,533
 castor_extractor/warehouse/synapse/queries/column.sql,sha256=lNcFoIW3Y0PFOqoOzJEXmPvZvfAsY0AP63Mu2LuPzPo,1351
-castor_extractor-0.24.9.dist-info/LICENCE,sha256=sL-IGa4hweyya1HgzMskrRdybbIa2cktzxb5qmUgDg8,8254
-castor_extractor-0.24.9.dist-info/METADATA,sha256=JDqbNB2dwsOO7_5PKUWP0r4FL217fi7OIEbVaOPljDQ,23985
-castor_extractor-0.24.9.dist-info/WHEEL,sha256=Nq82e9rUAnEjt98J6MlVmMCZb-t9cYE2Ir1kpBmnWfs,88
-castor_extractor-0.24.9.dist-info/entry_points.txt,sha256=FQNShG4w4nRO95_bZnagh7FQ2oiZ-40bdt8ZdTW1-uI,1731
-castor_extractor-0.24.9.dist-info/RECORD,,
+castor_extractor-0.24.10.dist-info/LICENCE,sha256=sL-IGa4hweyya1HgzMskrRdybbIa2cktzxb5qmUgDg8,8254
+castor_extractor-0.24.10.dist-info/METADATA,sha256=yB6KyU6Enp2Tcnmr4StHAmsKhwj3Fh0l7EnxaPQgM5c,24066
+castor_extractor-0.24.10.dist-info/WHEEL,sha256=Nq82e9rUAnEjt98J6MlVmMCZb-t9cYE2Ir1kpBmnWfs,88
+castor_extractor-0.24.10.dist-info/entry_points.txt,sha256=FQNShG4w4nRO95_bZnagh7FQ2oiZ-40bdt8ZdTW1-uI,1731
+castor_extractor-0.24.10.dist-info/RECORD,,