amoghrajesh commented on code in PR #66463:
URL: https://github.com/apache/airflow/pull/66463#discussion_r3217734827
##########
airflow-core/src/airflow/config_templates/config.yml:
##########
@@ -3025,6 +3025,24 @@ state_store:
type: string
example: "mypackage.state.CustomStateBackend"
default: "airflow.state.metastore.MetastoreStateBackend"
+ default_retention_days:
+ description: |
+ Number of days to retain task_state rows after their last update.
+ Rows older than this are removed by the scheduler's periodic cleanup.
+ This config does not affect asset_state rows.
Review Comment:
asset_state rows are cleaned up via foreign key cascade and are deleted when
the asset is deregistered — their lifecycle is tied to the asset, not a time
window. `default_retention_days` only applies to task_state because that's the
only one with time based expiry semantics. The description already calls this
out explicitly on the next line.
--
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]
