mykola-shyshov commented on issue #58146: URL: https://github.com/apache/airflow/issues/58146#issuecomment-3517663638
@Lee-W good progress. check, please, should we persist in db asset [extras](https://github.com/apache/airflow/blob/3c823ecd20d5ca347f4120c74b2fc64f4d6d8773/airflow-core/src/airflow/models/taskinstance.py#L1428) extras as well. and what may confuse as well is [warning](https://github.com/apache/airflow/blob/9ba09f3e07206bfcef7c9b6fff80b7185999a61e/airflow-core/src/airflow/assets/manager.py#L138). if we are okay to persist "dynamically" than warning is weird. Actually as for me not good method invocations (checks on None). maybe better to use something like `ensureAssetModel`. -- 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]
