nyoungstudios commented on PR #39772: URL: https://github.com/apache/airflow/pull/39772#issuecomment-2503342863
@TobyMellor Thanks for reaching out. My main problem is just having to resolve merge conflicts ever so often. This change was initially proposed in May of this year. The current status is that I need to resolve the merge conflicts. I have done so on this separate branch https://github.com/apache/airflow/compare/main...nyoungstudios:airflow:fix/serialize-trigger-dag-run-has-xcom-arg-new-version however, the now the link in the TriggerDagRunOperator doesn't work anymore. I'm getting this error in the scheduler: ``` airflow.exceptions.SerializationError: Failed to serialize DAG 'trigger_dag': '<' not supported between instances of 'str' and 'PlainXComArg' ``` I've yet to debug this problem, but I think it needs a change in the `DagDependency` class. Anyway, I'll try to eventually get around to fixing this again. But you are also free to submit a pr against my branch if you have a fix. I'm online on GitHub on all weekdays, so I can review it if you have a fix. -- 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]
