nathadfield commented on PR #63884:
URL: https://github.com/apache/airflow/pull/63884#issuecomment-4439656031

   > > One blocker:
   > > `airflow-core/src/airflow/ui/src/queries/useCreateBackfill.ts` drops 
`run_on_latest_version` before calling the generated mutation. 
`RunBackfillForm` resolves and submits the checkbox value into 
`createBackfill`, and the dry-run request includes it, but the actual create 
request only sends `dag_id`, dates, conf, max runs, reprocess behavior, and 
`run_backwards`. So if a user unchecks “Run on latest version” for a backfill 
with no Dag/global override, the backend receives `None` and falls back to 
`True` anyway. This breaks the explicit-request level of the hierarchy for the 
UI path.
   > > Please forward it in the mutation body, e.g. `run_on_latest_version: 
data.requestBody.run_on_latest_version`.
   > > Drafted-by: Codex (GPT-5); reviewed by @ephraimbuddy before posting
   > 
   > Has this been addressed?
   
   Good catch — fixed in 667e7ce. The dry-run was forwarding the whole 
requestBody so it worked, but the actual create explicitly enumerated fields 
and dropped run_on_latest_version. Added it to the mutation body so the 
explicit-request level of the hierarchy is now honored on the UI backfill path.


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