Skip to content

Commit b690929

Browse files
committed
Revert kernel telemetry helper guard
Signed-off-by: Jay Xiao <jay.xiao@databricks.com>
1 parent 40b0abc commit b690929

2 files changed

Lines changed: 0 additions & 11 deletions

File tree

src/databricks/sql/telemetry/telemetry_client.py

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -122,9 +122,6 @@ def get_auth_flow(auth_provider):
122122

123123
@staticmethod
124124
def is_telemetry_enabled(connection: "Connection") -> bool:
125-
if getattr(connection.session, "use_kernel", False) is True:
126-
return False
127-
128125
# Fast path: force enabled - skip feature flag fetch entirely
129126
if connection.force_enable_telemetry:
130127
return True

tests/unit/test_telemetry.py

Lines changed: 0 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -270,14 +270,6 @@ def test_token_federation_with_no_inner_provider(self):
270270
assert TelemetryHelper.get_auth_mechanism(fed) is None
271271
assert TelemetryHelper.get_auth_flow(fed) is None
272272

273-
def test_is_telemetry_enabled_returns_false_for_kernel(self):
274-
connection = MagicMock()
275-
connection.session.use_kernel = True
276-
connection.force_enable_telemetry = True
277-
connection.enable_telemetry = True
278-
279-
assert TelemetryHelper.is_telemetry_enabled(connection) is False
280-
281273

282274
class TestTelemetryFactory:
283275
"""Tests for TelemetryClientFactory lifecycle and management."""

0 commit comments

Comments
 (0)