docs(retry): correct the test comment and the stale README bullet

The test's ProtocolError with no cause is not a read timeout — those are
ReadTimeoutError and never reach that branch. Say what the value actually
is, and note that _is_connection_error is urllib3-private on an unpinned
transitive dependency.

README advertised configuring SDK retries; that is now the SDK's own
default, which this package deliberately leaves alone.

Author: anoop-narang

README.md

@@ -10,7 +10,7 @@ Runtime boundary and guarantees are defined in `CONTRACT.md`.
 
 - **Environment-driven client setup** — create clients from `HOTDATA_API_KEY`, optional `HOTDATA_API_URL`, and `HOTDATA_WORKSPACE`.
 - **Workspace resolution** — choose an explicit workspace from env, otherwise discover workspaces and select the active workspace or first available workspace.
-- **HTTP resilience** — configure SDK retries for transient connection failures and retry SQL execution on stale pooled sockets.
+- **HTTP resilience** — retry SQL execution on stale pooled sockets. Transport-level retries are the SDK's own default, which this package leaves in place so a non-idempotent request is never replayed on a response status.
 - **SQL execution helper** — run SQL through `POST /v1/query`, poll async query runs when needed, and return a `QueryResult`.
 - **Result utilities** — convert query results to records, pandas DataFrames, or metadata dictionaries for adapter display layers.
 - **History helpers** — list recent results and query run history with normalized dataclasses.

tests/test_retry_policy.py

@@ -48,9 +48,16 @@ def test_pre_response_connection_reset_still_retries_any_method() -> None:
     any response means the request never landed, so a POST retry is safe."""
     from urllib3.exceptions import ProtocolError
 
+    # `_is_connection_error` is urllib3-private, and urllib3 reaches this package
+    # only as an unpinned transitive dependency — so an upstream rename breaks
+    # this test for reasons unrelated to this repo. Accepted knowingly: it is the
+    # only hook that separates a reset from any other ProtocolError without
+    # standing up a live socket.
     retry = _retry_policy()
     cause = ConnectionResetError(54, "Connection reset by peer")
     reset = ProtocolError("Connection aborted.", cause)
     assert retry._is_connection_error(reset)
-    # A read timeout is NOT reclassified — the server may have done the work.
+    # A ProtocolError with no connection-level cause is NOT reclassified: only a
+    # reset proves the request never landed. Read timeouts never reach this branch
+    # at all — urllib3 raises those as ReadTimeoutError, and they stay method-gated.
     assert not retry._is_connection_error(ProtocolError("Connection aborted.", None))