hotdata-framework 0.6.1__tar.gz → 0.6.3__tar.gz

{hotdata_framework-0.6.1 → hotdata_framework-0.6.3}/CHANGELOG.md

@@ -8,11 +8,23 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 ## [Unreleased]
 
 
+## [0.6.3] - 2026-07-08
+
+### Added
+
+- `HotdataClient` and `ManagedDatabaseClient` accept `request_timeout` (seconds, or a `(connect, read)` pair). The generated SDK otherwise issues every HTTP request with urllib3's no-timeout default, so a stalled or unreachable server blocks the calling thread indefinitely; the new parameter applies a socket-level deadline to every call through the client while still honoring an explicit per-call `_request_timeout`. Also exported as `apply_default_request_timeout(api_client, timeout)` for callers holding a raw generated client. Default remains no timeout (behavior unchanged unless opted in).
+
+## [0.6.2] - 2026-07-08
+
+### Changed
+
+- Repository text cleanup: the changelog and test docstrings no longer reference external issue trackers. No functional changes; 0.6.2 is byte-identical to 0.6.1 in package code.
+
 ## [0.6.1] - 2026-07-08
 
 ### Fixed
 
-- `ManagedDatabaseClient.fetch_table` now carries the `X-Database-Id` scope header on the result poll, the query-run poll, and the Arrow fetch — not only on the query submit. Results of database-scoped queries are themselves database-scoped, so every read against an existing synced table (merge/append loads, dlt state restore) failed with `400: Bad Request` once the table had data (dlthubworker#70).
+- `ManagedDatabaseClient.fetch_table` now carries the `X-Database-Id` scope header on the result poll, the query-run poll, and the Arrow fetch — not only on the query submit. Results of database-scoped queries are themselves database-scoped, so every read against an existing synced table (merge/append loads, dlt state restore) failed with `400: Bad Request` once the table had data.
 - API error messages now include the response body (flattened, truncated to 500 chars). `400: Bad Request` alone hid the server's actual explanation.
 
 ### Changed

{hotdata_framework-0.6.1 → hotdata_framework-0.6.3}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: hotdata-framework
-Version: 0.6.1
+Version: 0.6.3
 Summary: Python framework for building Hotdata integrations: workspace/session runtime, query execution, and managed databases
 Project-URL: Homepage, https://www.hotdata.dev
 Project-URL: Documentation, https://www.hotdata.dev/docs

{hotdata_framework-0.6.1 → hotdata_framework-0.6.3}/hotdata_framework/client.py

@@ -1,5 +1,6 @@
 from __future__ import annotations
 
+import functools
 import time
 from collections.abc import Iterator
 from dataclasses import asdict, dataclass
@@ -71,6 +72,45 @@ class RunHistoryItem:
         return asdict(self)
 
 
+
+def apply_default_request_timeout(
+    api_client: ApiClient, timeout: float | tuple[float, float]
+) -> None:
+    """Give every request through this client a socket-level deadline.
+
+    The generated client forwards ``_request_timeout=None`` — urllib3's
+    no-timeout — on every call unless the caller passes one explicitly, and
+    most helper methods expose no such knob. A stalled or black-holed server
+    therefore blocks the calling thread indefinitely. Wrapping the REST seam
+    applies ``timeout`` (seconds, or a ``(connect, read)`` pair) as the
+    default while still honoring an explicit per-call ``_request_timeout``.
+    """
+    rest_client = api_client.rest_client
+    original = rest_client.request
+
+    @functools.wraps(original)
+    def request_with_default_timeout(
+        method,
+        url,
+        headers=None,
+        body=None,
+        post_params=None,
+        _request_timeout=None,
+    ):
+        if _request_timeout is None:
+            _request_timeout = timeout
+        return original(
+            method,
+            url,
+            headers=headers,
+            body=body,
+            post_params=post_params,
+            _request_timeout=_request_timeout,
+        )
+
+    rest_client.request = request_with_default_timeout
+
+
 class HotdataClient:
     """Thin wrapper around the Hotdata Python SDK with query polling helpers."""
 
@@ -81,6 +121,7 @@ class HotdataClient:
         *,
         host: str | None = None,
         session_id: str | None = None,
+        request_timeout: float | tuple[float, float] | None = None,
     ) -> None:
         self._host = normalize_host(host) if host else default_host()
         self._api_key = api_key
@@ -94,6 +135,8 @@ class HotdataClient:
             retries=default_http_retries(),
         )
         self._api = ApiClient(self._config)
+        if request_timeout is not None:
+            apply_default_request_timeout(self._api, request_timeout)
 
     @classmethod
     def from_env(cls) -> HotdataClient:

{hotdata_framework-0.6.1 → hotdata_framework-0.6.3}/hotdata_framework/managed_client.py

@@ -61,6 +61,7 @@ class ManagedDatabaseClient:
         api_base_url: str,
         max_retries: int,
         retry_backoff_seconds: float,
+        request_timeout: float | tuple[float, float] | None = None,
     ) -> None:
         self._max_retries = max_retries
         self._retry_backoff_seconds = retry_backoff_seconds
@@ -68,6 +69,7 @@ class ManagedDatabaseClient:
             api_key,
             workspace_id,
             host=api_base_url.rstrip("/"),
+            request_timeout=request_timeout,
         )
 
     def close(self) -> None:

{hotdata_framework-0.6.1 → hotdata_framework-0.6.3}/pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "hatchling.build"
 
 [project]
 name = "hotdata-framework"
-version = "0.6.1"
+version = "0.6.3"
 description = "Python framework for building Hotdata integrations: workspace/session runtime, query execution, and managed databases"
 readme = "README.md"
 requires-python = ">=3.10"

{hotdata_framework-0.6.1 → hotdata_framework-0.6.3}/tests/test_errors.py

@@ -1,8 +1,7 @@
 """Error message construction: the API's response body must survive.
 
 "400: Bad Request" alone is undebuggable; the body carries the server's
-actual explanation (e.g. which header was missing). Regression for the
-opaque load failures in dlthubworker#70.
+actual explanation (e.g. which header was missing).
 """
 
 from __future__ import annotations

{hotdata_framework-0.6.1 → hotdata_framework-0.6.3}/tests/test_managed_client.py

@@ -107,8 +107,7 @@ def test_fetch_table_carries_database_scope_on_result_reads(
     hotdata 0.6.0 SDK exposes ``x_database_id`` on all three.
 
     Regression: reruns/append loads against an existing synced table failed
-    with an opaque ``400: Bad Request`` (dlthubworker#70) because both reads
-    omitted the scope.
+    with an opaque ``400: Bad Request`` because both reads omitted the scope.
     """
     result_scopes: list[str | None] = []
     arrow_scopes: list[str | None] = []

hotdata_framework-0.6.3/tests/test_request_timeout.py

@@ -0,0 +1,69 @@
+"""The default request timeout must reach every call through the client.
+
+The generated client sends ``_request_timeout=None`` (urllib3 no-timeout)
+unless a caller passes one per call — and the helper methods expose no such
+knob — so a stalled server blocks the calling thread indefinitely.
+``apply_default_request_timeout`` wraps the REST seam once at construction.
+"""
+
+from __future__ import annotations
+
+from hotdata_framework.client import HotdataClient, apply_default_request_timeout
+from hotdata_framework.managed_client import ManagedDatabaseClient
+
+
+class _FakeRest:
+    def __init__(self) -> None:
+        self.seen: list[object] = []
+
+    def request(self, method, url, headers=None, body=None, post_params=None, _request_timeout=None):
+        self.seen.append(_request_timeout)
+        return "resp"
+
+
+class _FakeApi:
+    def __init__(self) -> None:
+        self.rest_client = _FakeRest()
+
+
+def test_default_applied_when_no_per_call_timeout() -> None:
+    api = _FakeApi()
+    rest = api.rest_client
+    apply_default_request_timeout(api, (10.0, 60.0))
+
+    assert api.rest_client.request("GET", "http://x") == "resp"
+    assert rest.seen == [(10.0, 60.0)]
+
+
+def test_explicit_per_call_timeout_wins() -> None:
+    api = _FakeApi()
+    rest = api.rest_client
+    apply_default_request_timeout(api, 60.0)
+
+    api.rest_client.request("GET", "http://x", _request_timeout=5.0)
+    assert rest.seen == [5.0]
+
+
+def test_hotdata_client_wraps_its_rest_seam() -> None:
+    client = HotdataClient("k", "w", host="https://example.test", request_timeout=7.0)
+    # The wrapper is installed in place of the generated bound method.
+    assert client.api.rest_client.request.__name__ == "request"
+    assert client.api.rest_client.request.__wrapped__ is not None
+
+
+def test_hotdata_client_without_timeout_is_untouched() -> None:
+    client = HotdataClient("k", "w", host="https://example.test")
+    assert not hasattr(client.api.rest_client.request, "__wrapped__")
+
+
+def test_managed_client_threads_timeout_through() -> None:
+    client = ManagedDatabaseClient(
+        api_key="k",
+        workspace_id="w",
+        api_base_url="https://example.test",
+        max_retries=1,
+        retry_backoff_seconds=0.0,
+        request_timeout=(5.0, 30.0),
+    )
+    assert client._runtime.api.rest_client.request.__wrapped__ is not None
+    client.close()

{hotdata_framework-0.6.1 → hotdata_framework-0.6.3}/uv.lock

@@ -101,7 +101,7 @@ wheels = [
 
 [[package]]
 name = "hotdata-framework"
-version = "0.6.1"
+version = "0.6.3"
 source = { editable = "." }
 dependencies = [
     { name = "hotdata" },