Default kernel telemetry to disabled

Signed-off-by: Jay Xiao <jay.xiao@databricks.com>

Author: jay-xiao446

src/databricks/sql/backend/kernel/client.py

@@ -191,7 +191,7 @@ def _kernel_telemetry_kwargs(options: Dict[str, Any]) -> Dict[str, Any]:
         # The Python telemetry model does not currently track process
         # name; omit it and let the kernel fill what it can derive.
         "process_name": None,
-        "telemetry_enabled": bool(options.get("enable_telemetry", True)),
+        "telemetry_enabled": bool(options.get("enable_telemetry", False)),
     }
     if options.get("telemetry_batch_size") is not None:
         out["telemetry_batch_size"] = options["telemetry_batch_size"]

src/databricks/sql/session.py

@@ -257,7 +257,7 @@ def _create_backend(
             # identity at Session construction time so kernel-owned
             # telemetry can populate its system configuration.
             kernel_telemetry_options = {
-                "enable_telemetry": kwargs.get("enable_telemetry", True),
+                "enable_telemetry": kwargs.get("enable_telemetry", False),
                 "telemetry_batch_size": kwargs.get("telemetry_batch_size"),
                 "telemetry_circuit_breaker_enabled": kwargs.get(
                     "_telemetry_circuit_breaker_enabled"

tests/unit/test_session.py

@@ -619,6 +619,42 @@ def test_telemetry_kwargs_threaded_into_kernel_client(self):
             finally:
                 conn.close()
 
+    def test_telemetry_enabled_defaults_false_for_kernel_client(self):
+        import sys
+        import types
+
+        pytest.importorskip(
+            "pyarrow",
+            reason="kernel client module imports pyarrow at load",
+        )
+
+        fake = types.ModuleType("databricks_sql_kernel")
+        fake.KernelError = type("KernelError", (Exception,), {})
+        fake.Session = MagicMock()
+
+        with patch.dict(sys.modules, {"databricks_sql_kernel": fake}), patch(
+            "databricks.sql.backend.kernel.client.KernelDatabricksClient"
+        ) as mock_kernel_client, patch(
+            "%s.session.get_python_sql_connector_auth_provider" % self.PACKAGE
+        ):
+            instance = mock_kernel_client.return_value
+            instance.open_session.return_value = SessionId(
+                BackendType.SEA, "sess-id", None
+            )
+
+            conn = databricks.sql.connect(
+                server_hostname="foo",
+                http_path="/sql/1.0/warehouses/abc",
+                use_kernel=True,
+                access_token="dapi-xyz",
+            )
+            try:
+                _, kwargs = mock_kernel_client.call_args
+                opts = kwargs["telemetry_options"]
+                assert opts["enable_telemetry"] is False
+            finally:
+                conn.close()
+
 
 class TestKernelUserAgentForwarding:
     """user_agent_entry must reach the kernel on the use_kernel path —

tests/unit/test_telemetry.py

@@ -332,6 +332,11 @@ def test_is_telemetry_enabled_returns_false_for_kernel(
             is expected_kernel_telemetry_enabled
         )
 
+    def test_kernel_telemetry_enabled_defaults_false(self):
+        kernel_kwargs = self._kernel_telemetry_kwargs_for_test({})
+
+        assert kernel_kwargs["telemetry_enabled"] is False
+
 
 class TestTelemetryFactory:
     """Tests for TelemetryClientFactory lifecycle and management."""