pierrejeambrun commented on PR #26252: URL: https://github.com/apache/airflow/pull/26252#issuecomment-1242008320
> On the dataset model we have `update_at` - can we use that here too please Good idea, renamed the fields to `updated_at`. > SQLAlchemy has a way of handling this natively that we should use instead https://docs.sqlalchemy.org/en/14/core/defaults.html?highlight=onupdate#context-sensitive-default-functions Done, used the `onupdate` param to keep this field updated. For now the field is 'nullable' because we do not have the information on existing records. Should I set it `nullable=False` and populate existing TaskInstance.updated_at and DagRun.updated_at with some default values ? -- 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]
