openlineage-python 1.24.0__tar.gz → 1.24.1__tar.gz

{openlineage_python-1.24.0 → openlineage_python-1.24.1}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.3
 Name: openlineage-python
-Version: 1.24.0
+Version: 1.24.1
 Summary: OpenLineage Python Client
 Author-email: OpenLineage <info@openlineage.io>
 Keywords: openlineage

{openlineage_python-1.24.0 → openlineage_python-1.24.1}/openlineage/client/client.py

@@ -82,7 +82,7 @@ class OpenLineageClient:
         transport: Transport | None = None,
         factory: TransportFactory | None = None,
         *,
-        config: dict[str, str] | None = None,
+        config: dict[str, Any] | None = None,
     ) -> None:
         # Set parent's logging level if environment variable is present
         custom_logging_level = os.getenv("OPENLINEAGE_CLIENT_LOGGING", None)
@@ -126,7 +126,15 @@ class OpenLineageClient:
 
     @classmethod
     def from_dict(cls: type[_T], config: dict[str, str]) -> _T:
-        return cls(config=config)
+        warnings.warn(
+            message=(
+                "Using `from_dict` to set transport is deprecated. "
+                "Use `config` parameter to fully configure OpenLineageClient."
+            ),
+            category=DeprecationWarning,
+            stacklevel=2,
+        )
+        return cls(transport=get_default_factory().create(config=config), config={"transport": config})
 
     def filter_event(
         self,

{openlineage_python-1.24.0 → openlineage_python-1.24.1}/openlineage/client/constants.py

@@ -2,7 +2,7 @@
 # SPDX-License-Identifier: Apache-2.0
 from __future__ import annotations
 
-__version__ = "1.24.0"
+__version__ = "1.24.1"
 
 DEFAULT_TIMEOUT_MS = 5000
 DEFAULT_NAMESPACE_NAME = "default"

{openlineage_python-1.24.0 → openlineage_python-1.24.1}/pyproject.toml

@@ -6,7 +6,7 @@ requires = [
 
 [project]
 name = "openlineage-python"
-version = "1.24.0"
+version = "1.24.1"
 description = "OpenLineage Python Client"
 readme = "README.md"
 keywords = [

{openlineage_python-1.24.0 → openlineage_python-1.24.1}/tests/test_client.py

@@ -402,8 +402,8 @@ def test_config(mocker: MockerFixture, root: Path) -> None:
 
 
 def test_openlineage_client_from_dict() -> None:
-    config_dict = {"transport": {"type": "http", "url": "http://localhost:5050"}}
-    client = OpenLineageClient.from_dict(config_dict)
+    transport_dict = {"type": "http", "url": "http://localhost:5050"}
+    client = OpenLineageClient.from_dict(transport_dict)
     assert client.transport.url == "http://localhost:5050"
 
 
@@ -680,7 +680,7 @@ def test_config_property_loads_user_defined_config() -> None:
 )
 def test_client_from_empty_dict_with_dynamic_env_vars() -> None:
     client = OpenLineageClient.from_dict({})
-    assert client.transport.url == "http://localhost:5050"
+    assert client.transport.kind == ConsoleTransport.kind
 
 
 @patch.dict(
@@ -691,7 +691,7 @@ def test_client_from_empty_dict_with_dynamic_env_vars() -> None:
 )
 def test_client_from_empty_dict_with_url_env_var() -> None:
     client = OpenLineageClient.from_dict({})
-    assert client.transport.url == "http://example.com"
+    assert client.transport.kind == ConsoleTransport.kind
 
 
 @patch.dict(
@@ -701,15 +701,14 @@ def test_client_from_empty_dict_with_url_env_var() -> None:
         "OPENLINEAGE__TRANSPORT__URL": "http://localhost:5050",
     },
 )
-def test_client_from_facets_config_in_dict_and_env_vars() -> None:
-    user_defined_config = {
+def test_client_raises_from_wrong_dict() -> None:
+    config_without_transport = {
         "facets": {
             "environment_variables": ["VAR1", "VAR2"],
         }
     }
-    client = OpenLineageClient.from_dict(user_defined_config)
-    assert client.config.facets.environment_variables == ["VAR1", "VAR2"]
-    assert client.transport.url == "http://localhost:5050"
+    with pytest.raises(KeyError):
+        OpenLineageClient.from_dict(config_without_transport)
 
 
 @patch.dict(
@@ -719,13 +718,11 @@ def test_client_from_facets_config_in_dict_and_env_vars() -> None:
     },
 )
 def test_client_from_facets_config_in_env_vars_and_transport_in_config() -> None:
-    user_defined_config = {
-        "transport": {
-            "type": "http",
-            "url": "http://localhost:5050",
-        }
+    transport_config = {
+        "type": "http",
+        "url": "http://localhost:5050",
     }
-    client = OpenLineageClient.from_dict(user_defined_config)
+    client = OpenLineageClient.from_dict(transport_config)
     assert client.config.facets.environment_variables == ["VAR1", "VAR2"]
     assert client.transport.url == "http://localhost:5050"
 
@@ -739,11 +736,9 @@ def test_client_from_facets_config_in_env_vars_and_transport_in_config() -> None
 @patch("openlineage.client.client.OpenLineageClient._find_yaml_config_path")
 @patch("openlineage.client.client.OpenLineageClient._get_config_file_content")
 def test_config_merge_precedence(mock_get_config_content, mock_find_yaml) -> None:
-    user_defined_config = {
-        "transport": {
-            "type": "http",
-            "url": "http://localhost:5050",
-        }
+    transport_config = {
+        "type": "http",
+        "url": "http://localhost:5050",
     }
     mock_find_yaml.return_value = "config.yml"
     mock_get_config_content.return_value = {
@@ -752,7 +747,7 @@ def test_config_merge_precedence(mock_get_config_content, mock_find_yaml) -> Non
             "auth": {"api_key": "another_token"},
         }
     }
-    client = OpenLineageClient.from_dict(user_defined_config)
+    client = OpenLineageClient.from_dict(transport_config)
     config = client.config
     assert config.transport["type"] == "http"
     assert config.transport["url"] == "http://localhost:5050"