FrankYang0529 commented on code in PR #68778:
URL: https://github.com/apache/airflow/pull/68778#discussion_r3505246276
##########
airflow-core/tests/unit/timetables/test_base_timetable.py:
##########
@@ -114,6 +114,58 @@ def
test_compute_rollup_fingerprint_window_change_produces_different_fingerprint
assert fp_hour != fp_day
+def test_compute_rollup_fingerprint_excludes_rerun_policy():
+ """
+ Changing only ``rerun_policy`` must NOT change the fingerprint.
+
+ ``rerun_policy`` governs post-fire behavior, not which upstream keys a
window
+ requires. If it fed the fingerprint, a policy-only edit would discard
in-flight
+ APDRs, and upgrading from a version that pre-dates the key would invalidate
+ every pending partition run (stored fingerprints lack the key).
+ """
+ from airflow.sdk import Asset, HourWindow, RerunPolicy, RollupMapper,
StartOfHourMapper
Review Comment:
Thank you. Updated it.
--
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]
