nathadfield commented on PR #63884: URL: https://github.com/apache/airflow/pull/63884#issuecomment-4269476640
Thanks @pierrejeambrun! I've made the following changes: - Moved the resolver to `services/public/common.py` as `resolve_run_on_latest_version`. It now takes a `fallback: bool = False` parameter so callers can pick the appropriate default for their context. - Backfill API endpoint now uses the resolver with `fallback=True` to preserve the historical default. `BackfillPostBody.run_on_latest_version` is now `bool | None = None` so callers who don't pass it get the full hierarchy resolution. - CLI backfill command now also uses the resolver with `fallback=True`. The `--run-on-latest-version` arg was changed from `store_true` to `BooleanOptionalAction (default None)`, so users can now pass `--run-on-latest-version` or `--no-run-on-latest-version` to explicitly override, or omit the flag to let the hierarchy resolve - Docs and newsfragment updated to mention backfills. - Tests added for the new fallback parameter and the case where DAG-level False overrides the fallback. Updated existing CLI tests to reflect the new resolver-based behavior. So now clear/rerun/backfill all use the same resolver with their own appropriate fallback, and the existing behaviors are preserved when no config or DAG-level override is set. -- 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]
