uranusjr commented on PR #65103: URL: https://github.com/apache/airflow/pull/65103#issuecomment-4294130695
_Technically_ it should be possible to keep the old syntax while still having the watcher asset-aware (set the watcher->asset relationship when a watcher is assigned to the asset), but the new way definitely makes the relationship clearer. I’m fine with ditching the old way. We need to figure out deprecation though. Do we only make the watcher asset-aware with the new syntax? What doest that mean for existing code? Only selected watchers would fail (cause a dag error at parse time) when it’s used with the old syntax? -- 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]
