jhtimmins commented on issue #14459: URL: https://github.com/apache/airflow/issues/14459#issuecomment-797714612
Thanks for the ping @XD-DENG. I think that `is_present` is too ambiguous (present in what? why would this change?). `in_last_dagbag_load` may be overly verbose, but I'd lean towards that or something similar to minimize the context needed to understand the variable. Regarding the filtering on "all active and paused DAGs" -- Looks like that logic was added in the original dag-specific permissions PR was added 3 years ago https://github.com/apache/airflow/commit/f3f2eb323f29d2210877ce9699e2dd0bdd8b5259. None of the design docs for this are accessible anymore, so I'm not sure what the reasoning was. My guess is that this was included so that active DAGs that were deleted through the UI don't stick around causing no-longer-relevant permissions to get created. ---------------------------------------------------------------- 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. For queries about this service, please contact Infrastructure at: [email protected]
