perf: Skip feature flag fetch when telemetry is force enabled

When force_enable_telemetry=True, skip the feature flag network call
entirely since we already know telemetry should be enabled.

This optimization:
- Eliminates unnecessary network call during connection init
- Faster connection setup when telemetry is forced on
- Clearer control flow with early returns

Author: samikshya-db

src/databricks/sql/telemetry/telemetry_client.py

@@ -110,18 +110,21 @@ def get_auth_flow(auth_provider):
 
     @staticmethod
     def is_telemetry_enabled(connection: "Connection") -> bool:
+        # Fast path: force enabled - skip feature flag fetch entirely
         if connection.force_enable_telemetry:
             return True
 
-        if connection.enable_telemetry:
-            context = FeatureFlagsContextFactory.get_instance(connection)
-            flag_value = context.get_flag_value(
-                TelemetryHelper.TELEMETRY_FEATURE_FLAG_NAME, default_value=False
-            )
-            return str(flag_value).lower() == "true"
-        else:
+        # Fast path: disabled - no need to check feature flag
+        if not connection.enable_telemetry:
             return False
 
+        # Only fetch feature flags when enable_telemetry=True and not forced
+        context = FeatureFlagsContextFactory.get_instance(connection)
+        flag_value = context.get_flag_value(
+            TelemetryHelper.TELEMETRY_FEATURE_FLAG_NAME, default_value=False
+        )
+        return str(flag_value).lower() == "true"
+
 
 class NoopTelemetryClient(BaseTelemetryClient):
     """