KoviAnusha commented on PR #58368: URL: https://github.com/apache/airflow/pull/58368#issuecomment-3539451655
Hey Mykola, great work on this fix. One question while reviewing: do we expect any noticeable perf impact when counting references for large asset sets? I assume minimal, but curious if we’ve thought about it. Also, confirming my understanding, this ensures inlet-only assets remain active and don’t get incorrectly dropped from AssetActive, correct? Maybe in the future we could add a test with mixed inlet/outlet references in the same DAG to fully cover real-world patterns, not necessary for this PR though. -- 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]
