eitan-shalev opened a new pull request, #72589:
URL: https://github.com/apache/airflow/pull/72589

   ### Description
   
   `BaseOperator.partial()` fills known BaseOperator arguments from framework 
defaults. If a custom operator overrides the signature default of one of those 
arguments, mapping currently materializes the framework default and then passes 
it into the operator during unmapping. That changes the behavior relative to 
direct task construction.
   
   Resolve defaulted BaseOperator arguments from the concrete operator 
signature before creating `partial_kwargs`. Explicit `.partial()` values and 
DAG/task default args retain their existing precedence.
   
   Adds a regression test using `retry_policy`, which is the newly exposed 
instance of this longstanding behavior.
   
   Closes: #72588
   
   ### Validation
   
   - Added a focused mapped/unmapped regression test.
   - Reproduced the behavior and verified the proposed resolution with Apache 
Airflow 3.3.1.
   - Full Task SDK CI is pending.
   
   ---
   
   ##### Was generative AI tooling used to co-author this PR?
   
   - [X] Yes: Codex
   
   Generated-by: Codex following [the 
guidelines](https://github.com/apache/airflow/blob/main/contributing-docs/05_pull_requests.rst#gen-ai-assisted-contributions)
   
   ---
   ๐Ÿ”— **Origin task in dev3:** [open in 
dev3](https://dev3.h0x91b.com/open.html?task=2e94cc8f-3e29-4dfb-87c6-e6ae10460cf5)
 ยท `dev3://task/2e94cc8f-3e29-4dfb-87c6-e6ae10460cf5`


-- 
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