Skip to content

Commit 02002a1

Browse files
ai: apply changes for #925 (1 review thread)
Addresses: - #3877271142 at src/databricks/sql/session.py:275 Signed-off-by: peco-engineer-bot[bot] <peco-engineer-bot[bot]@users.noreply.github.com>
1 parent 25879a1 commit 02002a1

1 file changed

Lines changed: 10 additions & 0 deletions

File tree

src/databricks/sql/client.py

Lines changed: 10 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -178,6 +178,16 @@ def __init__(
178178
``ImportError``. Supports PAT, OAuth M2M, and OAuth
179179
U2M auth, and native (positional and named) parameter
180180
binding. Mutually exclusive with ``use_sea``.
181+
182+
Telemetry note: on the kernel path telemetry is
183+
strictly opt-in. An explicit ``enable_telemetry=True``
184+
turns telemetry on unconditionally; it is NOT gated by
185+
the server-side ``enableTelemetryForPythonDriver``
186+
feature flag that applies on the Thrift/SEA paths. When
187+
``enable_telemetry`` is unset the kernel owns the enable
188+
decision. This is an intentional divergence from the
189+
Thrift/SEA paths, where an explicit ``True`` can still
190+
be suppressed by the feature flag.
181191
:param use_hybrid_disposition: `bool`, optional (default is False)
182192
Use the hybrid disposition instead of the inline disposition.
183193
:param server_hostname: Databricks instance host name.

0 commit comments

Comments
 (0)