Vamsi-klu opened a new pull request, #66682:
URL: https://github.com/apache/airflow/pull/66682

   Fix `TableauOperator(resource="tasks", method="run")` when Tableau tasks are 
triggered by id or by name.
   
   `Tasks.run()` expects a `TaskItem` and returns XML bytes, so calling it like 
other resources caused `AttributeError` when the operator tried to read `.id` 
from the raw response. This change resolves the task before calling `run()`, 
parses the returned job payload, and keeps existing behavior for other 
resources.
   
   Tests cover task lookup by id and by name, missing tasks, and empty task-run 
responses.
   
   closes: #42248
   
   Validation:
   - `breeze run --answer n --skip-image-upgrade-check pytest 
providers/tableau/tests/unit/tableau/operators/test_tableau.py -xvs`
   - `breeze run --answer n --skip-image-upgrade-check pytest 
providers/tableau/tests/unit -xvs`
   - `breeze run --answer n --skip-image-upgrade-check mypy 
providers/tableau/src/airflow/providers/tableau/operators/tableau.py`
   - `prek run --from-ref main --stage pre-commit`
   - `prek run --from-ref main --stage manual`
   
   ---
   
   ##### Was generative AI tooling used to co-author this PR?
   
   - [X] Yes — Codex GPT-5
   
   Generated-by: Codex GPT-5 following [the 
guidelines](https://github.com/apache/airflow/blob/main/contributing-docs/05_pull_requests.rst#gen-ai-assisted-contributions)
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to