sjyangkevin commented on PR #51511: URL: https://github.com/apache/airflow/pull/51511#issuecomment-3146572554
> @jason810496 is it ready to merge? Hi @eladkal , thanks for looking at it. I think it will be great if we could have a second eyes on it, as Jason mentioned in previous discussion. the caveat for this fix is that it runs the permission sync for ALL DAGs regardless those are updated or not. If we don’t check in this way, the current implementation will miss those permission error when dag processor run again, as discussed here https://github.com/apache/airflow/pull/51511#issue-3127821053 -- 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]
