Remove RUNNING+failed-task warning from execute()

The extra get_workflow() call on every RUNNING return adds an
unnecessary network round-trip. The underlying behavior (workflow stays
RUNNING while a failed task awaits retries) is correct Conductor
behavior, not a bug — tracked as a docs gap in
conductor-oss/getting-started#53.

Author: nthmost-orkes

src/conductor/client/workflow/executor/workflow_executor.py

@@ -1,10 +1,7 @@
 from __future__ import annotations
-import logging
 import uuid
 from typing import Any, Dict, List, Optional, TYPE_CHECKING
 
-logger = logging.getLogger(__name__)
-
 from typing_extensions import Self
 
 from conductor.client.configuration.configuration import Configuration
@@ -125,26 +122,12 @@ def execute(self, name: str, version: Optional[int] = None, workflow_input: Any
         if domain is not None:
             request.task_to_domain = {"*": domain}
 
-        workflow_run = self.workflow_client.execute_workflow(
+        return self.workflow_client.execute_workflow(
             start_workflow_request=request,
             request_id=request_id,
             wait_until_task_ref=wait_until_task_ref,
             wait_for_seconds=wait_for_seconds,
         )
-        if getattr(workflow_run, 'status', None) == 'RUNNING':
-            try:
-                full = self.workflow_client.get_workflow(workflow_run.workflow_id, include_tasks=True)
-                failed = [t for t in (full.tasks or []) if t.status in ('FAILED', 'FAILED_WITH_TERMINAL_ERROR')]
-                if failed:
-                    t = failed[0]
-                    reason = getattr(t, 'reason_for_incompletion', None) or 'see server logs'
-                    logger.warning(
-                        "Workflow %s returned RUNNING after timeout, but task '%s' has status %s: %s",
-                        workflow_run.workflow_id, t.reference_task_name, t.status, reason,
-                    )
-            except Exception:
-                pass
-        return workflow_run
 
     def remove_workflow(self, workflow_id: str, archive_workflow: Optional[bool] = None) -> None:
         """Removes the workflow permanently from the system"""