nathadfield commented on PR #61448: URL: https://github.com/apache/airflow/pull/61448#issuecomment-4037485746
I guess I can see three alternatives. **Option A: Consolidate into an enum** Replace both `disable_bundle_versioning` and `run_on_latest_version` with a single config like `bundle_version_strategy` with values: - **disabled**: no versioning at all (`current disable_bundle_versioning=True`) - **pinned**: use the version from when the run was created (current default) - **latest**: always use the latest version on clear/rerun (what `run_on_latest_version=True` does) This addresses the overlap concern directly and is arguably cleaner but has a big impact. **Option B: Extend disable_bundle_versioning** Rename/extend the existing flag to support a third state. E.g., `bundle_versioning = disabled | pinned | latest`. **Option C: Keep both, improve docs** Keep the current approach but add clear documentation explaining the distinction: `disable_bundle_versioning` turns off version tracking entirely (no version recorded), while `run_on_latest_version` keeps version tracking but controls the default rerun behavior. Add a comparison table. -- 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]
